From d9ef167e78e3e052813173e218a9747b81bea2ef Mon Sep 17 00:00:00 2001 From: Chafik Bel Date: Mon, 6 Nov 2017 13:38:54 +0100 Subject: [PATCH] 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. --- CODING_GUIDELINES.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/CODING_GUIDELINES.md b/CODING_GUIDELINES.md index 72afc7a2630..3282a24d728 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.