Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chore/dependency cleanup p3 #20946

Open
wants to merge 2 commits into
base: devel
Choose a base branch
from
Open

Conversation

goedderz
Copy link
Member

@goedderz goedderz commented May 16, 2024

Scope & Purpose

Deleted the non-local variant of AgencyCallbacks - they are unused and don't make sense any longer. Also got rid of ArangodServer as a dependency of AgencyComm, AgencyCallback, and AgencyCallbackRegistry. The latter as part of a larger undertaking to make features reusable again (note that features needing ArangodServer cannot be used without a complete ArangodServer, which makes them practically unusable for other tools or certain tests).

  • 馃敤 Refactoring/simplification

@goedderz goedderz added this to the devel milestone May 16, 2024
@goedderz goedderz self-assigned this May 16, 2024
@cla-bot cla-bot bot added the cla-signed label May 16, 2024
@goedderz goedderz marked this pull request as ready for review May 16, 2024 18:13
@goedderz goedderz requested review from a team as code owners May 16, 2024 18:13
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.

None yet

1 participant