Joshua Spence
bec13cfea8
Update PEP8.
...
Summary: Update `pep8.py` from v1.3.4 to v1.5.7. Because why not?
Test Plan: N/A
Reviewers: epriestley, #blessed_reviewers
Reviewed By: epriestley, #blessed_reviewers
Subscribers: epriestley, Korvin
Differential Revision: https://secure.phabricator.com/D9412
2014-06-07 11:31:26 -07:00
vrana
c7c3f6a7f1
Update PEP8 to 1.3.4
...
Summary: Primarily to avoid false positives: http://pypi.python.org/pypi/pep8#id1
Test Plan:
$ arc lint --engine ComprehensiveLintEngine externals/pep8/pep8.py # after uncommenting externals/ check
Saw 9 errors, haha.
Reviewers: epriestley
Reviewed By: epriestley
CC: aran, Korvin
Differential Revision: https://secure.phabricator.com/D4260
2012-12-21 11:34:40 -08:00
vrana
7133c76d37
Update PEP8
...
Test Plan:
Linted this Python file:
def get_mapper_sets(
compression=4.0, # Currently works for most tables
**kwargs):
pass
Didn't see:
> E225 missing whitespace around operator
Reviewers: andrewjcg, epriestley
Reviewed By: epriestley
CC: aran, Korvin
Differential Revision: https://secure.phabricator.com/D4172
2012-12-14 17:50:18 -08:00
Andrew Gallagher
399c505f1a
arc lint: fix/enable PEP8 linter
...
Summary:
This diff:
- Adds the PEP8 linter to the externals directory
- Changes the path for finding pep8.py
- Removes use of execx since pep8.py return an errors code
when it finds PEP8 violations
Test Plan:
tested linting python code
Reviewed By: epriestley
Reviewers: epriestley, jungejason
CC: aran, epriestley
Differential Revision: 309
2011-05-18 16:47:23 -07:00
