Skip to content

Commit

Permalink
DOC: Patch new flake8 command grep
Browse files Browse the repository at this point in the history
The grep was initially matching to "pandas,"
which is incorrect because that was also
matching files containing "pandas" in the
name but that were not in the main pandas
directory (e.g. performance test code)

Follow-up to pandas-devgh-15712

[ci skip]
  • Loading branch information
gfyoung committed Mar 20, 2017
1 parent 771e36c commit da57857
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion doc/source/contributing.rst
Original file line number Diff line number Diff line change
Expand Up @@ -527,7 +527,7 @@ unused function. However, style-checking the diff will not catch this because
the actual import is not part of the diff. Thus, for completeness, you should
run this command, though it will take longer::

git diff master --name-only -- '*.py' | grep 'pandas' | xargs -r flake8
git diff master --name-only -- '*.py' | grep 'pandas/' | xargs -r flake8

Backwards Compatibility
~~~~~~~~~~~~~~~~~~~~~~~
Expand Down

0 comments on commit da57857

Please sign in to comment.