8000 remove unused header files by jsteemann · Pull Request #11320 · arangodb/arangodb · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@jsteemann
Copy link
Contributor

Scope & Purpose

Remove 2 now unused header files

  • Bug-Fix for devel-branch (i.e. no need for backports?)
  • The behavior change can be verified via automatic tests

Testing & Verification

Test plan: compilation of arangod should still succeed

http://172.16.10.101:8080/view/PR/job/arangodb-matrix-pr/9127/

@jsteemann jsteemann added this to the devel milestone Mar 21, 2020
@jsteemann jsteemann requested review from ObiWahn and dothebart March 21, 2020 13:32
@jsteemann
Copy link
Contributor Author
jsteemann commented Mar 23, 2020

@jsteemann
Copy link
Contributor Author

Tests blue except unrelated hot_backup failure.

@jsteemann
Copy link
Contributor Author

@dothebart @ObiWahn : would you mind reviewing?

3 similar comments
@jsteemann
Copy link
Contributor Author

@dothebart @ObiWahn : would you mind reviewing?

@jsteemann
Copy link
Contributor Author

@dothebart @ObiWahn : would you mind reviewing?

@jsteemann
Copy link
Contributor Author

@dothebart @ObiWahn : would you mind reviewing?

@jsteemann jsteemann merged commit b7d021e into devel Mar 25, 2020
@jsteemann jsteemann deleted the bug-fix/remove-unused-headers branch March 25, 2020 12:12
@ObiWahn ObiWahn mentioned this pull request Mar 25, 2020
3 tasks
ObiWahn added a commit that referenced this pull request Mar 27, 2020
…ql-functions

* origin/devel:
  Bug fix/schema validation return code (#11341)
  Fix explainer output when restricting collections (#11338)
  remove tabstops
  Improve endpoint handling. (#11236)
  revert HTTP return code change for user API, add tests (#11331)
  remove unused header files (#11320)
  Feature/aql subquery execution block impl execute implementation batch sub queries (#11318)
  Fix isAdminUser. (#11326)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

0