forked from Pylons/pyramid
-
Notifications
You must be signed in to change notification settings - Fork 0
/
HACKING.txt
248 lines (168 loc) · 8.26 KB
/
HACKING.txt
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
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
Hacking on Pyramid
==================
Here are some guidelines for hacking on Pyramid.
Using a Development Checkout
----------------------------
You'll have to create a development environment to hack on Pyramid, using a
Pyramid checkout. You can either do this by hand, or if you have ``tox``
installed (it's on PyPI), you can use ``tox`` to set up a working development
environment. Each installation method is described below.
By Hand
+++++++
- While logged into your GitHub account, navigate to the Pyramid repo on
GitHub.
https://github.com/Pylons/pyramid
- Fork and clone the Pyramid repository to your GitHub account by clicking
the "Fork" button.
- Clone your fork of Pyramid from your GitHub account to your local computer,
substituting your account username and specifying the destination as
"hack-on-pyramid".
$ cd ~
$ git clone [email protected]:USERNAME/pyramid.git hack-on-pyramid
$ cd hack-on-pyramid
# Configure remotes such that you can pull changes from the Pyramid
# repository into your local repository.
$ git remote add upstream https://github.com/Pylons/pyramid.git
# fetch and merge changes from upstream into master
$ git fetch upstream
$ git merge upstream/master
Now your local repo is set up such that you will push changes to your GitHub
repo, from which you can submit a pull request.
- Create a virtual environment in which to install Pyramid:
$ cd ~/hack-on-pyramid
$ python3 -m venv env
From here on in within these instructions, the ``~/hack-on-pyramid/env``
virtual environment you created above will be referred to as ``$VENV``.
To use the instructions in the steps that follow literally, use the
``export VENV=~/hack-on-pyramid/env`` command.
- Install Pyramid from the checkout into the virtual environment, where the
current working directory is the ``pyramid`` checkout directory. We will
install Pyramid in editable (development) mode as well as its testing
requirements.
$ cd ~/hack-on-pyramid
$ $VENV/bin/pip install -e ".[testing,docs]"
- Optionally create a new Pyramid project using ``pcreate``:
$ cd $VENV
$ bin/pcreate -s starter starter
- ...and install the new project into the virtual environment:
$ cd $VENV/starter
$ $VENV/bin/pip install -e .
Using ``Tox``
+++++++++++++
Alternatively, if you already have ``tox`` installed, there is an easier
way to get going.
- Create a new directory somewhere and ``cd`` to it:
$ mkdir ~/hack-on-pyramid
$ cd ~/hack-on-pyramid
- Check out a read-only copy of the Pyramid source:
$ git clone git://github.com/Pylons/pyramid.git .
Alternatively, create a writeable fork on GitHub and clone it.
Since Pyramid is a framework and not an application, it can be convenient to
work against a sample application, preferably in its own virtual environment. A
quick way to achieve this is to use `tox
<http://tox.readthedocs.org/en/latest/>`_ with a custom configuration file
that is part of the checkout:
$ tox -c hacking-tox.ini
This will create a python-2.7 based virtual environment named ``env27``
(Pyramid's ``.gitconfig` ignores all top-level folders that start with ``env``
specifically in our use case), and inside that a simple pyramid application
named ``hacking`` that you can then fire up like so:
$ cd env27/hacking
$ ../bin/pip install -e ".[testing,docs]"
$ ../bin/pserve development.ini
Adding Features
---------------
In order to add a feature to Pyramid:
- The feature must be documented in both the API and narrative documentation
(in ``docs/``).
- The feature must work fully on the following CPython versions: 2.7, 3.4, 3.5,
3.6, and 3.7 on both UNIX and Windows.
- The feature must work on the latest version of PyPy.
- The feature must not depend on any particular persistence layer (filesystem,
SQL, etc).
- The feature must not add unnecessary dependencies (where "unnecessary" is of
course subjective, but new dependencies should be discussed).
The above requirements are relaxed for scaffolding dependencies. If a scaffold
has an install-time dependency on something that doesn't work on a particular
platform, that caveat should be spelled out clearly in *its* documentation
(within its ``docs/`` directory).
Coding Style
------------
- PEP8 compliance. Whitespace rules are relaxed: not necessary to put two
newlines between classes. But 79-column lines, in particular, are mandatory.
See https://pylonsproject.org/community-coding-style-standards.html for more
information.
- Please do not remove trailing whitespace. Configure your editor to reduce
diff noise. See https://github.com/Pylons/pyramid/issues/788 for more.
Running Tests
-------------
- To run all tests for Pyramid on a single Python version from your development
virtual environment (See *Using a Development Checkout* above), run
``nosetests``:
$ $VENV/bin/nosetests
- To run individual tests (i.e., during development), you can use ``nosetests``
syntax as follows:
# run a single test
$ $VENV/bin/nosetests pyramid.tests.test_module:ClassName.test_mytestname
# run all tests in a class
$ $VENV/bin/nosetests pyramid.tests.test_module:ClassName
Optionally you can install a nose plugin, `nose-selecttests
<https://pypi.org/project/nose-selecttests/>`_, and use a regular
expression with the ``-t`` parameter to run tests.
# run a single test
$ $VENV/bin/nosetests -t test_mytestname
- The ``tox.ini`` uses ``nose`` and ``coverage``. As such ``tox`` may be used
to run groups of tests or only a specific version of Python. For example, the
following command will run tests on Python 2.7 only without coverage:
$ tox -e py27
This command will run tests on the latest versions of Python 2 and 3 with
coverage totaled for both versions.
$ tox -e py2-cover,py3-cover,coverage
- To run the full set of Pyramid tests on all platforms, install `tox
<http://codespeak.net/~hpk/tox/>`_ into a system Python. The ``tox`` console
script will be installed into the scripts location for that Python. While
``cd``'ed to the Pyramid checkout root directory (it contains ``tox.ini``),
invoke the ``tox`` console script. This will read the ``tox.ini`` file and
execute the tests on multiple Python versions and platforms. While it runs,
it creates a virtual environment for each version/platform combination. For
example:
$ sudo /usr/bin/pip install tox
$ cd ~/hack-on-pyramid/
$ /usr/bin/tox
- The tests can also be run using `pytest <http://pytest.org/>`_. This is
intended as a convenience for people who are more used to or fond of
``pytest``. Run the tests like so:
$ $VENV/bin/pip install pytest
$ $VENV/bin/py.test --strict pyramid/
To run individual tests (i.e., during development), see "py.test usage -
Specifying tests / selecting tests":
http://pytest.org/latest/usage.html#specifying-tests-selecting-tests
- Functional tests related to the "scaffolds" (starter, zodb, alchemy) which
create a virtual environment, install the scaffold package and its
dependencies, start a server, and hit a URL on the server, can be run like
so:
$ ./scaffoldtests.sh
Alternatively:
$ tox -e{py27,py34,py35,pyt36,py37,pypy}-scaffolds
Test Coverage
-------------
- The codebase *must* have 100% test statement coverage after each commit. You
can test coverage via ``./coverage.sh`` (which itself just executes ``tox
-epy2-cover,py3-cover,coverage``).
Documentation Coverage and Building HTML Documentation
------------------------------------------------------
If you fix a bug, and the bug requires an API or behavior modification, all
documentation in this package which references that API or behavior must be
changed to reflect the bug fix, ideally in the same commit that fixes the bug
or adds the feature. To build and review docs, use the following steps.
1. In the main Pyramid checkout directory, run ``./builddocs.sh`` (which just
turns around and runs ``tox -e docs``):
$ ./builddocs.sh
2. Open the ``docs/_build/html/index.html`` file to see the resulting HTML
rendering.
Change Log
----------
- Feature additions and bugfixes must be added to the ``CHANGES.rst``
file in the prevailing style. Changelog entries should be long and
descriptive, not cryptic. Other developers should be able to know
what your changelog entry means.