8000 PEP8 testing by pelson · Pull Request #1945 · matplotlib/matplotlib · GitHub
[go: up one dir, main page]
Skip to content

PEP8 testing #1945

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
May 23, 2013
Merged

PEP8 testing #1945

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
fd57377
Added a coding_standards test module. Currently checks PEP8 complianc…
pelson Apr 25, 2013
bc3e3ed
Added some of the PEP8 guidelines to ignore in the coding standards t…
pelson May 14, 2013
bb1a328
Fixed failing PEP8 test.
pelson May 14, 2013
747b849
PEP8 test only on python2. 2to3 produces non pep8 compliant code.
pelson May 21, 2013
bd4f6fd
Fixed tests after rebase.
pelson May 21, 2013
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Added some of the PEP8 guidelines to ignore in the coding standards t… 
…est.
  • Loading branch information
@pelson
pelson committed May 21, 2013
commit bc3e3ed097ca646845925a3bef4b35628f4ebc0e
20 changes: 5 additions & 15 deletions lib/matplotlib/tests/test_coding_standards.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,40 +23,28 @@ class StandardReportWithExclusions(pep8.StandardReport):
'*/matplotlib/bezier.py',
'*/matplotlib/cbook.py',
'*/matplotlib/collections.py',
'*/matplotlib/colorbar.py',
'*/matplotlib/dates.py',
'*/matplotlib/docstring.py',
'*/matplotlib/dviread.py',
'*/matplotlib/finance.py',
'*/matplotlib/font_manager.py',
'*/matplotlib/fontconfig_pattern.py',
'*/matplotlib/gridspec.py',
'*/matplotlib/hatch.py',
'*/matplotlib/image.py',
'*/matplotlib/legend.py',
'*/matplotlib/legend_handler.py',
'*/matplotlib/mathtext.py',
'*/matplotlib/mlab.py',
'*/matplotlib/nxutils.py',
'*/matplotlib/offsetbox.py',
'*/matplotlib/patches.py',
'*/matplotlib/path.py',
'*/matplotlib/patheffects.py',
'*/matplotlib/pylab.py',
'*/matplotlib/pyplot.py',
'*/matplotlib/quiver.py',
'*/matplotlib/rcsetup.py',
'*/matplotlib/sankey.py',
'*/matplotlib/scale.py',
'*/matplotlib/stackplot.py',
'*/matplotlib/streamplot.py',
'*/matplotlib/table.py',
'*/matplotlib/texmanager.py',
'*/matplotlib/text.py',
'*/matplotlib/textpath.py',
'*/matplotlib/ticker.py',
'*/matplotlib/tight_bbox.py',
'*/matplotlib/tight_layout.py',
'*/matplotlib/transforms.py',
'*/matplotlib/type1font.py',
'*/matplotlib/widgets.py',
Expand All @@ -72,7 +60,6 @@ class StandardReportWithExclusions(pep8.St A020 andardReport):
'*/matplotlib/testing/jpl_units/UnitDblConverter.py',
'*/matplotlib/testing/jpl_units/UnitDblFormatter.py',
'*/matplotlib/testing/jpl_units/__init__.py',
'*/matplotlib/tri/triangulation.py',
'*/matplotlib/tri/tricontour.py',
'*/matplotlib/tri/triinterpolate.py',
'*/matplotlib/tri/tripcolor.py',
Expand Down Expand Up @@ -147,8 +134,6 @@ class StandardReportWithExclusions(pep8.StandardReport):
'*/matplotlib/backends/windowing.py',
'*/matplotlib/backends/qt4_editor/figureoptions.py',
'*/matplotlib/backends/qt4_editor/formlayout.py',
'*/matplotlib/delaunay/interpolate.py',
'*/matplotlib/delaunay/testfuncs.py',
'*/matplotlib/sphinxext/__init__.py',
'*/matplotlib/sphinxext/ipython_console_highlighting.py',
'*/matplotlib/sphinxext/ipython_directive.py',
Expand Down Expand Up @@ -202,6 +187,11 @@ def test_pep8_conformance():
pep8style = pep8.StyleGuide(quiet=False,
reporter=StandardReportWithExclusions)

# Extend the number of PEP8 guidelines which are not checked.
pep8style.options.ignore = pep8style.options.ignore + ('E121', 'E122',
'E123', 'E124', 'E125', 'E126', 'E127',
'E128')

# Allow users to add their own exclude list.
extra_exclude_file = os.path.join(os.path.dirname(__file__),
'.pep8_test_exclude.txt')
Expand Down
0