-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Code Freeze] Restructuring the code #1473
Labels
enhancement
New feature or request
Comments
For clarity, this issue is tagged [Code Freeze] as the maintainers will wait to merge any non-reorg changes until after the re-org? Makes sense to me but wanted to clarify |
5 tasks
@kev-the-dev, I realise that I have not actually written that out. Thanks for asking. Yes, we are not taking contributions at the moment. |
This was referenced Sep 13, 2023
This was referenced Sep 24, 2023
Closed
Closed
This has been implemented, and we are preparing to get these changes out as alpha soon, closing. |
This was referenced Sep 28, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Juan and I are currently undertaking a large code refactor to make the code more maintainable.
Currently the goal is to split a lot of the code into modules, which forces us to have separate concerns and make sure that more of the code does one thing, and does it well. We had a lot of cycling dependencies (allowed by one large package) and it made everything very tightly knitted, and testing very hard as you had to set up a massive dependency chain.
Rough plan for how we want to do it:
hscontrol
(Code reorg step 1 #1444)types
contains all our datatypes, and functions attached to them without dependenciesdb
contains all functions that manipulate and changes the databasepolicy
contains all functions related to ACLs, a big change is that it does no longer depend ondb
mapper
, all code related to generating MapResponses (the map clients need to find their peers)(For now, leave grpc in root)api
, the code that implements our gRPC interfacederp
, derp related codedns
, related codeThe text was updated successfully, but these errors were encountered: