summaryrefslogtreecommitdiff
path: root/doc
diff options
context:
space:
mode:
authorlouiz’ <louiz@louiz.org>2018-09-01 23:12:39 +0200
committerlouiz’ <louiz@louiz.org>2018-09-01 23:12:39 +0200
commite75b551a52f0f426b3bc098318d66ec9c75fc9a1 (patch)
treea71d033a7cbea0803e99aa5f3154fb25ddb5c1f8 /doc
parentb9fab4aafb97748d7842b4074c647d2f1ad78e7a (diff)
downloadbiboumi-e75b551a52f0f426b3bc098318d66ec9c75fc9a1.tar.gz
biboumi-e75b551a52f0f426b3bc098318d66ec9c75fc9a1.tar.bz2
biboumi-e75b551a52f0f426b3bc098318d66ec9c75fc9a1.tar.xz
biboumi-e75b551a52f0f426b3bc098318d66ec9c75fc9a1.zip
Add the contributing.rst doc in the index
Diffstat (limited to 'doc')
-rw-r--r--doc/contributing.rst106
-rw-r--r--doc/index.rst1
2 files changed, 107 insertions, 0 deletions
diff --git a/doc/contributing.rst b/doc/contributing.rst
new file mode 100644
index 0000000..ee499fc
--- /dev/null
+++ b/doc/contributing.rst
@@ -0,0 +1,106 @@
+#######################
+Contributing to biboumi
+#######################
+
+Biboumi’s main workplace is at https://lab.louiz.org/louiz/biboumi
+
+The repository is also mirrored on other websites, for example on github,
+but that’s mainly for the convenience of users.
+
+Before doing anything, you can come on the `XMPP chatroom`_ to discuss your
+changes, issues or ideas.
+
+
+Bug reports, feature requests
+-----------------------------
+
+To open a bug report, or a feature request, please do so on `our gitlab’s
+bug tracker`_.
+
+If the bug you’re reporting is about a bad behaviour of biboumi when some XMPP
+or IRC events occur, please try to reproduce the issue with a biboumi running
+in log_level=0, and include the relevant logs in your bug report.
+
+If the issue you’re reporting may have security implications, please
+select the “confidential” flag in your bug report. This includes, but is not limited to:
+
+- disclosure of private data that was supposed to be encrypted using TLS
+- denial of service (crash, infinite loop, etc) that can be caused by any
+ user
+
+
+Code
+----
+
+To contribute code, you can do so using git: commit your changes on any
+publicly available git repository and communicate us its address. This can
+be done with a `gitlab merge request`_, or a `github pull request`_ or just
+by sending a message into the `XMPP chatroom`_.
+
+It is suggested that you use gitlab’s merge requests: this will
+automatically run our continuous integration tests.
+
+It is also recommended to add some unit or end-to-end tests for the proposed
+changes.
+
+
+Tests
+-----
+
+There are two test suites for biboumi:
+
+- unit tests that can be run simply using `make check`.
+ These tests use the Catch test framework, are written in pure C++
+ and they should always succeed, in all possible build configuration.
+
+- a more complex end-to-end test suite. This test suite is written in python3,
+ uses a specific IRC server (`charybdis`_), and only tests the most complete
+ biboumi configuration (when all dependencies are used). To run it, you need
+ to install various dependencies: refer to fedora’s `Dockerfile.base`_ and
+ `Dockerfile`_ to see how to install charybdis, slixmpp, botan, a ssl
+ certificate, etc.
+
+ Once all the dependencies are correctly installed, the tests are run with
+
+.. code-block:: sh
+
+ make e2e
+
+To run one or more specific tests, you can do something like this:
+
+.. code-block:: sh
+
+ make biboumi && python3 ../tests/end_to_end self_ping basic_handshake_success
+
+This will run two tests, self_ping and basic_handshake_success.
+
+To write additional tests, you need to add a Scenario
+into `the __main__.py file`_. If you have problem running this end-to-end
+test suite, or if you struggle with this weird code (that would be
+completely normal…), don’t hesitate to ask for help.
+
+
+All these tests automatically run with various configurations, on various
+platforms, using gitlab CI.
+
+
+Coding style
+------------
+Please try to follow the existing style:
+
+- Use only spaces, not tabs.
+- Curly brackets are on their own lines.
+- Use this-> everywhere it’s possible.
+- Don’t start class attributes with “m_” or similar.
+- Type names are in PascalCase.
+- Everything else is in snake_case.
+
+
+.. _our gitlab’s bug tracker: https://lab.louiz.org/louiz/biboumi/issues/new
+.. _gitlab merge request: https://lab.louiz.org/louiz/biboumi/merge_requests/new
+.. _github pull request: https://github.com/louiz/biboumi/pulls
+.. _XMPP chatroom: xmpp:biboumi@muc.poez.io
+.. _Dockerfile.base: docker/biboumi-test/fedora/Dockerfile.base
+.. _Dockerfile: docker/biboumi-test/fedora/Dockerfile
+.. _charybdis: https://github.com/charybdis-ircd/charybdis
+.. _the __main__.py file: tests/end_to_end/__main__.py
diff --git a/doc/index.rst b/doc/index.rst
index ca1bf0d..b97c8fd 100644
--- a/doc/index.rst
+++ b/doc/index.rst
@@ -20,3 +20,4 @@ XMPP client as if these channels were XMPP MUCs.
install
admin
user
+ contributing