You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@spark.apache.org by "Sean Owen (JIRA)" <ji...@apache.org> on 2017/09/19 20:08:00 UTC

[jira] [Created] (SPARK-22066) Update checkstyle to 8.2, enable it, fix violations

Sean Owen created SPARK-22066:
---------------------------------

             Summary: Update checkstyle to 8.2, enable it, fix violations
                 Key: SPARK-22066
                 URL: https://issues.apache.org/jira/browse/SPARK-22066
             Project: Spark
          Issue Type: Improvement
          Components: Build
    Affects Versions: 2.3.0
            Reporter: Sean Owen
            Assignee: Sean Owen
            Priority: Minor


While working on Scala 2.12 changes, I noted we could update various build plugins to their latest version, such as the scala-maven-plugin.

While doing that I noticed that the checkstyle plugin config had some bogus config, and that it was actually disabled and could be enabled now (build prints ERRORs but doesn't fail). This noise was making it a little hard to figure out what errors are real and new in the Scala 2.12 changes I'm working on, so should be fixed and stay fixed by being treated as errors.

And then I noted we could update checkstyle itself to 8.2, and fix existing and additional style errors.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@spark.apache.org
For additional commands, e-mail: issues-help@spark.apache.org