pystache icon indicating copy to clipboard operation
pystache copied to clipboard

Make compliant with PEP 8

Open cjerdonek opened this issue 13 years ago • 1 comments

We should also add to setup.py and/or the tests the ability to run the PEP 8 checks.

cjerdonek avatar Jul 16 '12 00:07 cjerdonek

This could potentially be accomplished with relative ease using the setuptools-pep8 module. Although this module does not appear to be actively developed, its maturity and rate of download are enough assurance for me that it's reliable enough.

The following modifications would need to be made:

# setup.py

requires = []

. . .

# Add the 'setuptools-pep8' module to the required
# modules list.
requires.append('setuptools-pep8')  
# setup.cfg

[pep8]
ignore=E225

The pep8 check would be run by invoking python setup.py pep8.

Now obviously more could be done to improve upon this, as it basically spits out a disgusting mess of lint errors currently. The above is merely my findings from some quick experimentation.

Anyways, just my $0.02.

jessebraham avatar Dec 02 '14 17:12 jessebraham