diff options
author | Chafik Bel <chafik.bel@gmail.com> | 2017-11-06 13:38:54 +0100 |
---|---|---|
committer | Dag Wieers <dag@wieers.com> | 2017-11-06 13:38:54 +0100 |
commit | d9ef167e78e3e052813173e218a9747b81bea2ef (patch) | |
tree | e9c744f9bd4c28d9c2a7f16a2bbc9f0a2f59307f /CODING_GUIDELINES.md | |
parent | dc40efc13f2b344235ff668a2c3c6b235219f239 (diff) | |
download | ansible-d9ef167e78e3e052813173e218a9747b81bea2ef.tar.gz |
PEP8 set the line limit (#32578)
Highlight that the line length is limited to 160 characters now, since the test will fail if the line is longer than 160 chars.
Diffstat (limited to 'CODING_GUIDELINES.md')
-rw-r--r-- | CODING_GUIDELINES.md | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/CODING_GUIDELINES.md b/CODING_GUIDELINES.md index 72afc7a263..3282a24d72 100644 --- a/CODING_GUIDELINES.md +++ b/CODING_GUIDELINES.md @@ -15,11 +15,11 @@ Language * If you want to write non-Python ansible modules or inventory scripts, that's fine, but they are not going to get merged in most likely. Sorry!! PEP 8 and basic style checks -=========================== +============================ * [PEP 8](https://www.python.org/dev/peps/pep-0008/) is a great Python style guide, which you should read. * PEP 8 must not be strictly followed in all aspects, but most of it is good advice - * In particular, we don't really care about line lengths. Buy a bigger monitor! + * The line is limited to 160 characters. * To run checks for things we care about, use [ansible-test](http://docs.ansible.com/ansible/dev_guide/testing_pep8.html#running-locally). * Similarly, additional checks can be made with "make pyflakes" * There is no need to submit code changes for PEP 8 and pyflakes fixes, as these break attribution history. Project leadership will make these periodically. |