Laurens Van Houtven | 290aba1 | 2014-03-14 15:20:18 +0100 | [diff] [blame] | 1 | # Contributing |
| 2 | |
| 3 | First of all, thank you for your interest in contributing to |
Laurens Van Houtven | dc06b70 | 2014-03-15 18:11:47 +0100 | [diff] [blame^] | 4 | pyOpenSSL! |
Laurens Van Houtven | 290aba1 | 2014-03-14 15:20:18 +0100 | [diff] [blame] | 5 | |
| 6 | ## Filing bug reports |
| 7 | |
| 8 | Bug reports are very welcome. Please file them on the Github issue |
| 9 | tracker. Good bug reports come with extensive descriptions of the |
| 10 | error and how to reproduce it. Reporters are strongly encouraged to |
| 11 | include an [SSCCE](http://www.sscce.org/). |
| 12 | |
| 13 | ## Patches |
| 14 | |
Laurens Van Houtven | dc06b70 | 2014-03-15 18:11:47 +0100 | [diff] [blame^] | 15 | All patches to pyOpenSSL should be submitted in the form of pull |
| 16 | requests to the main pyOpenSSL repository, `pyca/pyopenssl`. These |
Laurens Van Houtven | 290aba1 | 2014-03-14 15:20:18 +0100 | [diff] [blame] | 17 | pull requests should satisfy the following properties: |
| 18 | |
| 19 | - Pull requests that involve code must follow the |
| 20 | [Twisted Coding Standard][tcs]. For example, `methodNamesLikeThis`, |
| 21 | three empty lines between module-level elements, and two empty lines |
| 22 | between class-level elements. |
| 23 | - Pull requests that introduce code must test all new behavior they |
| 24 | introduce, as well as previously untested or poorly tested behavior |
| 25 | that they touch. |
| 26 | - Pull requests are not allowed to break existing tests. |
| 27 | - Pull requests that include major changes should note those changes |
| 28 | in the `ChangeLog` text file in the root of the repository. |
| 29 | |
| 30 | Finally, pull requests must be reviewed before merging. This process |
| 31 | is based on [the one from cryptography][cryptography-review]. Everyone |
| 32 | can perform reviews; this is a very valuable way to contribute, and is |
| 33 | highly encouraged. |
| 34 | |
| 35 | All members of the pyca Github organization can merge pull requests, |
| 36 | of course keeping in mind all the requirements detailed in this |
| 37 | document as well as the pyca/cryptography merge requirements. |
| 38 | |
| 39 | The final responsibility for the reviewing of merged code lies with |
Laurens Van Houtven | dc06b70 | 2014-03-15 18:11:47 +0100 | [diff] [blame^] | 40 | the person merging it; since pyOpenSSL is obviously a sensitive |
Laurens Van Houtven | 290aba1 | 2014-03-14 15:20:18 +0100 | [diff] [blame] | 41 | project from a security perspective, so reviewers are strongly |
| 42 | encouraged to take this review and merge process very seriously. |
| 43 | |
| 44 | [tcs]: https://twistedmatrix.com/documents/current/core/development/policy/coding-standard.html |
| 45 | "Twisted Coding Standard" |
| 46 | [cryptography-review]: https://cryptography.io/en/latest/development/reviewing-patches/ |