diff options
author | drebs <drebs@leap.se> | 2017-07-12 15:46:15 -0300 |
---|---|---|
committer | drebs <drebs@leap.se> | 2017-07-12 15:46:15 -0300 |
commit | b367b0ff86297e8bbb4c7c01dc3a705234e31d2f (patch) | |
tree | 47d11db00fe8c2ce4f93217c83e93d6d5b1b8f0e | |
parent | f70b1d00446864d2a2a21bb6b2a954b4c9b20729 (diff) |
[doc] add a contributing page
-rw-r--r-- | docs/contributing.rst | 48 | ||||
-rw-r--r-- | docs/development.rst | 1 |
2 files changed, 49 insertions, 0 deletions
diff --git a/docs/contributing.rst b/docs/contributing.rst new file mode 100644 index 00000000..76e7450c --- /dev/null +++ b/docs/contributing.rst @@ -0,0 +1,48 @@ +Contributing +============ + +Thank you for your interest in contributing to Soledad! + +Filing bug reports +------------------ + +Bug reports are very welcome. Please file them on the `0xacab issue tracker +<https://0xacab.org/leap/soledad/issues>`_. Please include an extensive +description of the error, the steps to reproduce it, the version of Soledad +used and the provider used (if applicable). + +Patches +------- + +All patches to Soledad should be submitted in the form of pull requests to the +`main Soledad repository <https://0xacab.org/leap/soledad>`_. These pull +requests should satisfy the following properties: + +* The pull request should focus on one particular improvement to Soledad. + Create different pull requests for unrelated features or bugfixes. +* Code should follow `PEP 8 <https://www.python.org/dev/peps/pep-0008/>`_, + especially in the "do what code around you does" sense. +* Pull requests that introduce code must test all new behavior they introduce + as well as for previously untested or poorly tested behavior that they touch. +* Pull requests are not allowed to break existing tests. We will consider pull + requests that are breaking the CI as work in progress. +* When introducing new functionality, please remember to write documentation. +* Please sign all of your commits. If you are merging code from other + contributors, you should sign their commits. + +Review +~~~~~~ + +Pull requests must be reviewed before merging. The final responsibility for the +reviewing of merged code lies with the person merging it. Exceptions to this +rule are modifications to documentation, packaging, or tests when the need of +agility of merging without review has little or no impact on the security and +functionality of working code. + +Getting help +~~~~~~~~~~~~ + +If you need help you can reach us through one of the following means: + +* IRC: ``#leap`` at ``irc.freenode.org``. +* Mailing list: `leap-discuss@lists.riseup.net <https://lists.riseup.net/www/info/leap-discuss>`_ diff --git a/docs/development.rst b/docs/development.rst index 2a4303e0..d90e9db0 100644 --- a/docs/development.rst +++ b/docs/development.rst @@ -7,5 +7,6 @@ should help you get started with development for Soledad. .. toctree:: :maxdepth: 1 + contributing tests benchmarks |