|
1 | 1 | # Contributing
|
2 | 2 |
|
3 |
| -When contributing to this repository, please first discuss the change you wish to make via issue, email, or any other method with the owners of this repository before making a change. |
| 3 | +- [Contributing](#contributing) |
| 4 | + - [Preferred contributions](#preferred-contributions) |
| 5 | + - [Pull Request Process](#pull-request-process) |
| 6 | + - [Developer's guide](#developers-guide) |
| 7 | + - [Tests Units](#tests-units) |
| 8 | + |
| 9 | +When contributing to this repository, please first discuss the change you wish to make via issue with the owners of this repository before making a change. |
4 | 10 |
|
5 | 11 | Please note we have a code of conduct, please follow it in all your interactions with the project.
|
6 | 12 |
|
| 13 | +## Preferred contributions |
| 14 | + |
| 15 | +At the moment, these kind of contributions are more appreciated and should be preferred: |
| 16 | + |
| 17 | +- Fix for issues described in [Known Issues](./README.md#known-issues) |
| 18 | +- New cool features. Please propose your features in an issue first. |
| 19 | +- Code optimizations: any optimization to the code is welcome |
| 20 | + |
| 21 | +For any other kind of contribution, especially for new features, please submit an issue first. |
| 22 | + |
7 | 23 | ## Pull Request Process
|
8 | 24 |
|
9 |
| -1. Ensure any install or build dependencies are removed before the end of the layer when doing a build. |
10 |
| -2. Update the README.md with details of changes to the interface, this includes new line in the changelog, new modules if added, new command line options if added, etc. |
11 |
| -3. Increase the version numbers in any examples files and the README.md to the new version that this Pull Request would represent. The versioning scheme we use is [SemVer](http://semver.org/). |
12 |
| -4. You may merge the Pull Request in once you have the sign-off of two other developers, or if you do not have permission to do that, you may request the second reviewer to merge it for you. |
| 25 | +1. Write your code. |
| 26 | +2. Write a **properly documentation** compliant with **reStructuredText**. |
| 27 | +3. Write tests for your code. **Code coverage for your code must be at least at 90%**. See [Tests Units](#tests-units) to know more! |
| 28 | +4. Report changes to the issue you opened. |
| 29 | +5. Update the README.md and other docs with details of changes to the interface, this includes new line in the changelog, new modules if added, new command line options if added, etc. |
| 30 | +6. Request maintainers to merge your changes. |
| 31 | + |
| 32 | +## Developer's guide |
| 33 | + |
| 34 | +Want to know more about how ATtila works? |
| 35 | +Check out the [Developers guide](docs/devwiki.md) |
| 36 | + |
| 37 | +## Tests Units |
| 38 | + |
| 39 | +ATtila is provided with tests units, which can be found under tests/ directory. |
| 40 | +To launch test unit just type: |
| 41 | + |
| 42 | +```sh |
| 43 | +nosetests -v --with-coverage --cover-tests --cover-package=attila --nocapture tests/ |
| 44 | +``` |
| 45 | + |
| 46 | +Don't you have nose? |
| 47 | + |
| 48 | +```sh |
| 49 | +sudo -H pip3 install nose coverage unittest2 codecov |
| 50 | +``` |
| 51 | + |
| 52 | +--- |
13 | 53 |
|
14 |
| -Thank you for any contribution! |
| 54 | +Thank you for any contribution! 🧡 |
15 | 55 | Christian Visintin
|
0 commit comments