Opened 17 years ago
Closed 17 years ago
#5601 closed (fixed)
The Coding Style section in Contributing Docs should mention PEP8 checkers
Reported by: | simeon | Owned by: | simeon |
---|---|---|---|
Component: | Documentation | Version: | dev |
Severity: | Keywords: | pep8 code style | |
Cc: | Triage Stage: | Ready for checkin | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
I just discovered pep8.py at http://svn.browsershots.org/trunk/devtools/pep8/ and consequently removed some style issues from a patch I submitted that I wouldn't have seen otherwise (whitespace and trailing semi-colons). Perhaps more experienced Python devs are familiar with this or similar tools but I thought it might not be a bad idea to point to it in the docs. I just added the text
(use `pep8.py`_ to check).
to the end of the line about PEP 8 in the Coding Style section of the contributing.txt doc page (and appended the link to the bottom of the page).
Attachments (1)
Change History (3)
by , 17 years ago
comment:1 by , 17 years ago
Keywords: | pep8 code style added |
---|---|
Triage Stage: | Unreviewed → Ready for checkin |
comment:2 by , 17 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
I agree--pep8.py is very useful and it can't hurt to point it out when talking about coding style.