summaryrefslogtreecommitdiff
path: root/doc/source/dev/contributing.rst
blob: 8d386c87cd53295cbf95da460098b4a3af35bb35 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
Contributing
============

Conventions
-----------

We don’t have a strict set of conventions, but you should respect PEP8 mostly
(e.g. 4 spaces, class names in CamelCase and methods lowercase with
underscores) except if it means less-readable code (80 chars is often a hassle,
and if you look inside poezio you’ll see lots of long lines, mostly because of
strings).

As explained in the :ref:`overview`, “global” code goes in
:file:`core.py`, tab-related code goes in :file:`tabs.py`, and ui-related code goes in
:file:`windows.py`. There are other modules (e.g. :file:`xhtml.py`) but they do not matter
for the application as a whole.

Commit guidelines
-----------------

Commits **should** have a meaningful title (first line), and *may* have a detailed
description below. There are of course exceptions (for example, a single-line
commit that takes care of a typo right behind a big commit does not need to
say ``fix a typo ("azre" → "are") in toto.py line 45454``, since the metainfos
already take care of that.), but if you do not have commit access on the
poezio trunk, you can still reset and commit again.


Try to do atomic commits: since git is a DVCS, it doesn’t hurt to ``git add -p``
and split the commit into several meaningful small commits ; on the contrary,
it helps to track the changes on different levels.


If you have a conflict, solve it with rebase and not merge if the fast-forwards
do not resolve it automatically in your case. This helps to avoid creating
useless merges (and polluting the commit history) when none is needed.

.. code-block:: bash

    git fetch origin
    git rebase origin/master
    git push origin master

If your commit is related to an issue on our tracker_ (or fixes such an
issue), you can use ``Fix #BUGID`` or ``References #BUGID`` to help with the
tracking.


Getting your code into poezio
-----------------------------

If you have code you want to contribute, you can:

* Give us a patch and a description of what it does
* Give us a link to a **git** repo from which we can pull

The code is of course reviewed and tested a bit, but we trust the contributors
to submit good code. If we can’t integrate the given code into poezio (if it
crashes or has some issues), if the size is small, we may tweak it ourselves
and integrate it, and if not, you are of course free to take our advice into
account and submit it again.


If you have already submitted some code and plan to do more, you can ask us
 direct commit access on the main repo.

.. _tracker: https://dev.louiz.org/project/poezio/bugs