We're always looking to improve this project! Open source contribution is encouraged so long as they adhere to these guidelines.
The Flagsmith team will be monitoring for pull requests. When we get one, a member of team will test the work against our internal uses and sign off on the changes. From here, we'll either merge the pull request or provide feedback suggesting the next steps.
A couple things to keep in mind
- If you've changed APIs, update the documentation.
- Keep the code style (indents, wrapping) consistent.
- If your PR involves a lot of commits, squash them using
git rebase -ias this makes it easier for us to review.
- Keep lines under 80 characters.
The application uses pre-commit configuration (
.pre-commit-config.yaml ) to run
formatting before commits.
To install pre-commit:
# From the repository root
You can also manually run all the checks across the entire codebase with:
pre-commit run --all-files
The application uses pytest for writing (appropriate use of fixtures) and running tests. Before running tests please
make sure that
DJANGO_SETTINGS_MODULE env var is pointing to the right module, e.g.
To run tests: