Uploaded image for project: 'Spark'
  1. Spark
  2. SPARK-2627

Check for PEP 8 compliance on all Python code in the Jenkins CI cycle

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 1.1.0
    • Build
    • None

    Description

      This issue was triggered by the discussion here.

      Requirements:

      • make a linter script for Scala under dev/lint-scala that just calls scalastyle
      • make a linter script for Python under dev/lint-python that calls pep8 on all Python files
      • Modify dev/run-tests to call both linter scripts
      • Incorporate these changes into the Contributing to Spark guide

      Attachments

        There are no Sub-Tasks for this issue.

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            nchammas Nicholas Chammas
            nchammas Nicholas Chammas
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment