summaryrefslogtreecommitdiff
path: root/CODING_GUIDELINES.md
diff options
context:
space:
mode:
authorChafik Bel <chafik.bel@gmail.com>2017-11-06 13:38:54 +0100
committerDag Wieers <dag@wieers.com>2017-11-06 13:38:54 +0100
commitd9ef167e78e3e052813173e218a9747b81bea2ef (patch)
treee9c744f9bd4c28d9c2a7f16a2bbc9f0a2f59307f /CODING_GUIDELINES.md
parentdc40efc13f2b344235ff668a2c3c6b235219f239 (diff)
downloadansible-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.md4
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.