Skip to content

Commit

Permalink
PEP8 set the line limit (ansible#32578)
Browse files Browse the repository at this point in the history
Highlight that the line length is limited to 160 characters now, since the test will fail if the line is longer than 160 chars.
  • Loading branch information
Chafik Bel authored and dagwieers committed Nov 6, 2017
1 parent dc40efc commit d9ef167
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions CODING_GUIDELINES.md
Original file line number Diff line number Diff line change
Expand Up @@ -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.
Expand Down

0 comments on commit d9ef167

Please sign in to comment.