Repository: spark
Updated Branches:
  refs/heads/master 7fdacbc77 -> 544a1ba67


[SPARK-22375][TEST] Test script can fail if eggs are installed by set…

…up.py during test process

## What changes were proposed in this pull request?

Ignore the python/.eggs folder when running lint-python

## How was this patch tested?
1) put a bad python file in python/.eggs and ran the original script.  results 
were:

xins-MBP:spark xinlu$ dev/lint-python
PEP8 checks failed.
./python/.eggs/worker.py:33:4: E121 continuation line under-indented for 
hanging indent
./python/.eggs/worker.py:34:5: E131 continuation line unaligned for hanging 
indent

2) test same situation with change:

xins-MBP:spark xinlu$ dev/lint-python
PEP8 checks passed.
The sphinx-build command was not found. Skipping pydoc checks for now

Author: Xin Lu <x...@salesforce.com>

Closes #19597 from xynny/SPARK-22375.


Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/544a1ba6
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/544a1ba6
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/544a1ba6

Branch: refs/heads/master
Commit: 544a1ba678810b331d78fe9e63c7bb2342ab3d99
Parents: 7fdacbc
Author: Xin Lu <x...@salesforce.com>
Authored: Sun Oct 29 15:29:23 2017 +0900
Committer: hyukjinkwon <gurwls...@gmail.com>
Committed: Sun Oct 29 15:29:23 2017 +0900

----------------------------------------------------------------------
 dev/tox.ini | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/544a1ba6/dev/tox.ini
----------------------------------------------------------------------
diff --git a/dev/tox.ini b/dev/tox.ini
index eeeb637..eb8b1eb 100644
--- a/dev/tox.ini
+++ b/dev/tox.ini
@@ -16,4 +16,4 @@
 [pep8]
 ignore=E402,E731,E241,W503,E226
 max-line-length=100
-exclude=cloudpickle.py,heapq3.py,shared.py,python/docs/conf.py,work/*/*.py
+exclude=cloudpickle.py,heapq3.py,shared.py,python/docs/conf.py,work/*/*.py,python/.eggs/*


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to