Comment 0 for bug 483934

Revision history for this message
Jonathan Lange (jml) wrote :

It would be nice if Quickly generated code that conforms to the normal standard for Python code: PEP 8.

The ubuntu-project templates at the least are not PEP 8 compliant.

$ find . -name '*.py' | xargs /home/jml/bin/pep8.py
./setup.py:7:80: E501 line too long (89 characters)
./setup.py:26:25: W291 trailing whitespace
./setup.py:67:5: E301 expected 1 blank line, found 0
./setup.py:92:56: E202 whitespace before ')'
./setup.py:94:1: W391 blank line at end of file
./lister/listerconfig.py:18:1: E302 expected 2 blank lines, found 1
./lister/PreferencesListerDialog.py:59:48: E231 missing whitespace after ':'
./lister/PreferencesListerDialog.py:63:95: E225 missing whitespace around operator

Also, docstrings should have a short first line, a blank line and then extended text, as per PEP 257.

http://www.python.org/dev/peps/pep-0008/
http://www.python.org/dev/peps/pep-0257/

I personally would mark this as a really low priority issue and suggest that the person filing the ticket provide a patch, but that's just me.