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
72
73
74
75
76
77
78
79
80
81
82
83
|
===================================
Notes for contributing to testtools
===================================
Coding style
------------
In general, follow PEP 8 <http://www.python.org/dev/peps/pep-0008/>.
For consistency with the standard library's ``unittest`` module, method names
are generally ``camelCase``.
testtools supports Python 2.4 and later, so avoid any 2.5-only features like
the ``with`` statement.
Copyright assignment
--------------------
Currently all code in testtools is copyright Jonathan M. Lange. For the sake
of licensing simplicity, copyright of contributed code needs to be assigned to
Jonathan M. Lange otherwise it cannot be accepted. Please include an
appropriate statement of copyright assignment with contributions.
Testing
-------
Please write tests for every feature. This project ought to be a model
example of well-tested Python code!
Take particular care to make sure the *intent* of each test is clear.
You can run tests with ``make check``, or by running ``./run-tests`` directly.
Source layout
-------------
The top-level directory contains the ``testtools/`` package directory, and
miscellaneous files like README and setup.py.
The ``testtools/`` directory is the Python package itself. It is separated
into submodules for internal clarity, but all public APIs should be “promoted”
into the top-level package by importing them in ``testtools/__init__.py``.
Users of testtools should never import a submodule, they are just
implementation details.
Tests belong in ``testtools/tests/``.
Commiting to trunk
------------------
Testtools is maintained using bzr, with its trunk at lp:testtools. This gives
every contributor the ability to commit their work to their own branches.
However permission must be granted to allow contributors to commit to the trunk
branch.
Commit access to trunk is obtained by joining the testtools-devs Launchpad
team. Membership in this team is contingent on obeying the testtools
contribution policy, including assigning copyright of all the work one creates
and places in trunk to Jonathan Lange.
Code Review
-----------
All code must be reviewed before landing on trunk. The process is to create a
branch in launchpad, and submit it for merging to lp:testtools. It will then
be reviewed before it can be merged to trunk. It will be reviewed by someone:
* not the author
* a committer (member of the testtools-devs team)
As a special exception, while the testtools committers team is small and prone
to blocking, a merge request from a committer that has not been reviewed after
24 hours may be merged by that committer. When the team is larger this policy
will be revisited.
Code reviewers should look for the quality of what is being submitted,
including conformance with this HACKING file.
Changes which all users should be made aware of should be documented in NEWS.
|