This repository uses trunk based development. The picture below depicts what you must follow to properly contribute to this project:
Follow the steps below if you want to contribute with opentelemetry-instrumentation-django-stomp
:
- Firstly, you should search in issues something that you want to work with. You could filter those with
help wanted
(Good for community contributors to help, up-for-grabs) orgood first issue
labels. If you don't find issue with your contribution idea you can create one issue or open a discussion to explain your idea or requirement for contribution; - To contribute, you must create a new branch for
main
from this repo; - Then, you must create the code following the Code style and push them in your branch created previously;
- After all commits, you should open a PR from your branch to the
main
branch;- The PR go trigger a pipeline with test, sonar and lint validation, if all checks are OK a test version package is generate and published in https://test.pypi.org/project/opentelemetry-instrumentation-django-stomp/;
- The Maintainers will use this package to validate the incoming code;
- Maintainers will review your pull request ASAP, and they can approve it or request updates;
- Maintainers will merge the PR and a new version of package will be generated;
Another way to contribute is creating an issues or discussion. Feel free to write as many issues as you want, they are very important to help us to launch new releases and solving bugs. It's recommended to label you issue with any one of the following labels: question
, bug
or enhancement
.
Make sure to run all the tests before opening a pull request. Any new feature should also be tested!. You can run the tests with docker or tox
docker-compose up integration-tests
Or using tox
pipenv run tox
The project use .pre-commit-config.yaml
of flake8, black, isort and pylint.You can run the .pre-commit-config.yaml
with docker
docker-compose up lint-formatter
Or using pre-commit
pre-commit run --all-files
The pull request will run sonar which may show some flags and these should be resolved. Is expected 90% of coverage in new codes.
The commit
summary should be structured as conventional commits standard.