diff options
author | Tristan Van Berkom <tristan.vanberkom@codethink.co.uk> | 2019-01-03 15:53:27 -0500 |
---|---|---|
committer | Tristan Van Berkom <tristan.vanberkom@codethink.co.uk> | 2019-01-03 16:04:49 -0500 |
commit | 91c8b634e92af722ce6f41d7448e2e1fe9aa7640 (patch) | |
tree | 84115e0bc10f882cd16ccffa512a87618b580fe7 | |
parent | 9caa0ea0405c7d98b6955424b7c8503b9bd2dda1 (diff) | |
download | buildstream-91c8b634e92af722ce6f41d7448e2e1fe9aa7640.tar.gz |
CONTRIBUTING.rst: Adding instructions to run the linter separatelytristan/toxic-linting
-rw-r--r-- | CONTRIBUTING.rst | 21 |
1 files changed, 12 insertions, 9 deletions
diff --git a/CONTRIBUTING.rst b/CONTRIBUTING.rst index 0079cb4b5..ddcc4130d 100644 --- a/CONTRIBUTING.rst +++ b/CONTRIBUTING.rst @@ -1498,6 +1498,18 @@ option when running tox:: tox -e py37 +Linting is performed separately from testing. In order to run the linting step which +consists of running the ``pycodestyle`` and ``pylint`` tools, run the following:: + + tox -e lint + +.. tip:: + + The project specific pylint and pycodestyle configurations are stored in the + toplevel buildstream directory in the ``.pylintrc`` file and ``setup.cfg`` files + respectively. These configurations can be interesting to use with IDEs and + other developer tooling. + The output of all failing tests will always be printed in the summary, but if you want to observe the stdout and stderr generated by a passing test, you can pass the ``-s`` option to pytest as such:: @@ -1532,15 +1544,6 @@ output. To run them, you can use:: tox -- --integration -By default, buildstream also runs pylint on all files. Should you want -to run just pylint (these checks are a lot faster), you can do so -with:: - - tox -- -m pylint - -Alternatively, any IDE plugin that uses pytest should automatically -detect the ``.pylintrc`` in the project's root directory. - In case BuildStream's dependencies were updated since you last ran the tests, you might see some errors like ``pytest: error: unrecognized arguments: --codestyle``. If this happens, you |