blob: 3a818fcfd4537da585cde000043bed9012dddd13 (
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
68
69
70
71
|
===================
Python Coding style
===================
Coding style
~~~~~~~~~~~~
:ref:`black` is the tool used to reformat the Python code.
Linting
~~~~~~~
The Python linting is done by :ref:`Flake8` and :ref:`pylint`
They are executed by mozlint both at review phase and in the CI.
Indentation
~~~~~~~~~~~
Four spaces in Python code.
Makefile/moz.build practices
----------------------------
- Changes to makefile and moz.build variables do not require
build-config peer review. Any other build system changes, such as
adding new scripts or rules, require review from the build-config
team.
- Suffix long ``if``/``endif`` conditionals with #{ & #}, so editors
can display matched tokens enclosing a block of statements.
::
ifdef CHECK_TYPE #{
ifneq ($(flavor var_type),recursive) #{
$(warning var should be expandable but detected var_type=$(flavor var_type))
endif #}
endif #}
- moz.build are python and follow normal Python style.
- List assignments should be written with one element per line. Align
closing square brace with start of variable assignment. If ordering
is not important, variables should be in alphabetical order.
.. code-block:: python
var += [
'foo',
'bar'
]
- Use ``CONFIG['CPU_ARCH'] {=arm}`` to test for generic classes of
architecture rather than ``CONFIG['OS_TEST'] {=armv7}`` (re: bug 886689).
Other advices
~~~~~~~~~~~~~
- Install the
`mozext <https://hg.mozilla.org/hgcustom/version-control-tools/file/default/hgext/mozext>`__
Mercurial extension, and address every issue reported on commit
or the output of ``hg critic``.
- Follow `PEP 8 <https://www.python.org/dev/peps/pep-0008/>`__. Please run :ref:`black` for this.
- Do not place statements on the same line as ``if/elif/else``
conditionals to form a one-liner.
- Global vars, please avoid them at all cost.
- Exclude outer parenthesis from conditionals.Use
``if x > 5:,``\ rather than ``if (x > 5):``
- Use string formatters, rather than var + str(val).
``var = 'Type %s value is %d'% ('int', 5).``
- Testing/Unit tests, please write them and make sure that they are executed in the CI.
|
