Skip to content

Commit 44e3e56

Browse files
committed
[SPARK-2627] use tox.ini to exclude files
No need to exclude files in the call to pep8. The tool references tox.ini for config information, so we should use that.
1 parent b09fae2 commit 44e3e56

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

dev/lint-python

+1-2
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,7 @@ fi
4444
#+ first, but we do so so that the check status can
4545
#+ be output before the report, like with the
4646
#+ scalastyle and RAT checks.
47-
python $PEP8_SCRIPT_PATH ./python --exclude="cloudpickle.py" \
48-
> "$PEP8_REPORT_PATH"
47+
python $PEP8_SCRIPT_PATH ./python > "$PEP8_REPORT_PATH"
4948
pep8_status=${PIPESTATUS[0]} #$?
5049

5150
if [ $pep8_status -ne 0 ]; then

tox.ini

+1
Original file line numberDiff line numberDiff line change
@@ -15,3 +15,4 @@
1515

1616
[pep8]
1717
max-line-length=100
18+
exclude=cloudpickle.py

0 commit comments

Comments
 (0)