Skip to content

[SPARK-9385] [PYSPARK] Enable PEP8 but disable installing pylint.#7704

Closed
yhuai wants to merge 1 commit into
apache:masterfrom
yhuai:SPARK-9385
Closed

[SPARK-9385] [PYSPARK] Enable PEP8 but disable installing pylint.#7704
yhuai wants to merge 1 commit into
apache:masterfrom
yhuai:SPARK-9385

Conversation

@yhuai

@yhuai yhuai commented Jul 27, 2015

Copy link
Copy Markdown
Contributor

Instead of disabling all python style check, we should enable PEP8. So, this PR just comments out the part installing pylint.

@yhuai

yhuai commented Jul 27, 2015

Copy link
Copy Markdown
Contributor Author

Alright, I saw

========================================================================
Running Python style checks
========================================================================
PEP8 checks passed.

in the jenkins output. I am merging it to master.

@asfgit asfgit closed this in dafe8d8 Jul 27, 2015
@SparkQA

SparkQA commented Jul 28, 2015

Copy link
Copy Markdown

Test build #38596 has finished for PR 7704 at commit 0056359.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants