diff options
| author | Jason Robinson <jasonr@matrix.org> | 2019-11-01 16:45:09 +0200 |
|---|---|---|
| committer | Amber Brown <hawkowl@atleastfornow.net> | 2019-11-02 01:45:09 +1100 |
| commit | 67a65918ad7723400f25339ef3f4447e7b3dc1b6 (patch) | |
| tree | 236661baba6c8437c2cee47386365271f2b4525c /CONTRIBUTING.rst | |
| parent | Support for routing outbound HTTP requests via a proxy (#6239) (diff) | |
| download | synapse-67a65918ad7723400f25339ef3f4447e7b3dc1b6.tar.xz | |
Add contributer docs for using the provided linters script (#6164)
* Add lint dependencies black, flake8 and isort These are required when running the `lint.sh` dev scripts. Signed-off-by: Jason Robinson <jasonr@matrix.org> * Add contributer docs for using the providers linters script Add also to the pull request template to avoid build failures due to people not knowing that linters need running. Signed-off-by: Jason Robinson <jasonr@matrix.org> * Fix mention of linter errors correction Co-Authored-By: Andrew Morgan <1342360+anoadragon453@users.noreply.github.com> * Add mention for installing linter dependencies Co-Authored-By: Andrew Morgan <1342360+anoadragon453@users.noreply.github.com> * Remove linters from python dependencies as per PR review Signed-off-by: Jason Robinson <jasonr@matrix.org>
Diffstat (limited to 'CONTRIBUTING.rst')
| -rw-r--r-- | CONTRIBUTING.rst | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/CONTRIBUTING.rst b/CONTRIBUTING.rst index 2fb3a95949..df81f6e54f 100644 --- a/CONTRIBUTING.rst +++ b/CONTRIBUTING.rst @@ -70,6 +70,17 @@ Please ensure your changes match the cosmetic style of the existing project, and **never** mix cosmetic and functional changes in the same commit, as it makes it horribly hard to review otherwise. +Before doing a commit, ensure the changes you've made don't produce +linting errors. You can do this by running the linters as follows. Ensure to +commit any files that were corrected. + +:: + # Install the dependencies + pip install -U black flake8 isort + + # Run the linter script + ./scripts-dev/lint.sh + Changelog ~~~~~~~~~ |