-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add CONTRIBUTING.md #810
Comments
@batpad is this ticket still relevant? |
Hey @nanometrenat Thanks for the reminder :D. Yes, it is still relevant. Sample configs for future reference.
The changes will change most of the codebase, so we will need to make these changes after all PRs have been merged to avoid change conflicts. |
Hi peeps! Is this still a thing? Not sure if some of the older PRs in this repo will ever be merged at this stage. |
Hey @nanometrenat, Thanks for checking in We've made some updates in PR #2128, which this depends on. We will start working on this soon. Furthermore, we can close this one #811 now |
Breaking out of conversation here: #806 (comment)
We should add more comprehensive developer documentation to the backend repository, as we have more people working on the code. This should cover code-style guidelines, setting up linters, writing tests.
There's a bit of clean-up that would ideally be done before this makes sense, to ensure the current code follows the guidelines.
@thenav56 @gulfaraz
The text was updated successfully, but these errors were encountered: