3 commits

Author	SHA1	Message	Date
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