From 454a661d444dfc3e97db826159edc2b35ee820e7 Mon Sep 17 00:00:00 2001 From: Liam Marshall Date: Fri, 9 Oct 2015 15:46:03 -0500 Subject: [PATCH] contributing.rst: fix nits pointed out by @kuba --- docs/contributing.rst | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/docs/contributing.rst b/docs/contributing.rst index a76d76cd8..b4b1619ce 100644 --- a/docs/contributing.rst +++ b/docs/contributing.rst @@ -269,14 +269,19 @@ Please: Submitting a pull request ========================= + Steps: 1. Write your code! -2. Make sure your environment is set up properly and that you're in your virtualenv (this is a **very important** step). -3. Run ``./pep8.travis.sh`` to do a cursory check of your code style. Fix any errors. +2. Make sure your environment is set up properly and that you're in your + virtualenv. You can do this by running ``./bootstrap/dev/venv.sh``. + (this is a **very important** step) +3. Run ``./pep8.travis.sh`` to do a cursory check of your code style. + Fix any errors. 4. Run ``tox -e lint`` to check for pylint errors. Fix any errors. -5. Run ``tox`` to run the unit tests. Fix any errors. -6. Run the integration tests, see `integration`_. +5. Run ``tox`` to run the entire test suite including coverage. Fix any errors. +6. If your code touches communication with an ACME server/Boulder, you + should run the integration tests, see `integration`_. 7. Submit the PR. Updating the documentation