1
1
# Contributing
2
2
3
3
When contributing to this repository, please first discuss the change you wish to make via issue,
4
- email, or any other method with the maintainers of this repository before making a change.
4
+ email, or any other method with the maintainers of this repository before making a change.
5
5
6
6
Please note we have a code of conduct, please follow it in all your interactions with the project.
7
7
8
8
## Pull Request Process
9
9
10
- 1 . Ensure any install or build dependencies are removed before the end of the layer when doing a
10
+ 1 . Ensure any install or build dependencies are removed before the end of the layer when doing a
11
11
build.
12
12
2 . This project uses flake8 to conform with common Python standards. Make sure
13
13
to run your code through linter using latest version of flake8, before pull request.
@@ -16,11 +16,11 @@ Please note we have a code of conduct, please follow it in all your interactions
16
16
4 . pytest is used for automated testing. Please make sure to update tests that are needed, and to run
17
17
` make test ` before submitting your pull request. This should prevent issues with TravisCI and
18
18
make the review and merging process easier and faster.
19
- 5 . Update the README.md with details of changes to the interface, this includes new environment
19
+ 5 . Update the README.md with details of changes to the interface, this includes new environment
20
20
variables, exposed ports, useful file locations and container parameters.
21
21
6 . Increase the version numbers in any examples files and the README.md to the new version that this
22
22
Pull Request would represent. The versioning scheme we use is [ SemVer] ( http://semver.org/ ) .
23
- 7 . You may merge the Pull Request in once you have the sign-off of one other developer. If you
23
+ 7 . You may merge the Pull Request in once you have the sign-off of one other developer. If you
24
24
do not have permission to do that, you may request reviewer to merge it for you.
25
25
26
26
## Decorum
0 commit comments