From b367b0ff86297e8bbb4c7c01dc3a705234e31d2f Mon Sep 17 00:00:00 2001 From: drebs Date: Wed, 12 Jul 2017 15:46:15 -0300 Subject: [doc] add a contributing page --- docs/contributing.rst | 48 ++++++++++++++++++++++++++++++++++++++++++++++++ docs/development.rst | 1 + 2 files changed, 49 insertions(+) create mode 100644 docs/contributing.rst (limited to 'docs') 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 +`_. 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 `_. 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 `_, + 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 `_ 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 -- cgit v1.2.3