You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@storm.apache.org by "Stig Rohde Døssing (JIRA)" <ji...@apache.org> on 2018/11/11 16:29:00 UTC

[jira] [Commented] (STORM-2891) Upgrade Checkstyle plugin to version 3.0.0

    [ https://issues.apache.org/jira/browse/STORM-2891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16682927#comment-16682927 ] 

Stig Rohde Døssing commented on STORM-2891:
-------------------------------------------

Primarily this is to be able to output the violations as "plain" rather than XML. I find the plain format much more readable.

Sample:
{code}
Starting audit...
[WARN] /mnt/c/Users/Esran/Documents/NetbeansProjects/storm/storm-client/src/jvm/org/apache/storm/annotation/InterfaceStability.java:21: Javadoc comment at column 5 has parse error. Details: no viable alternative at input '</ul' while parsing HTML_ELEMENT [NonEmptyAtclauseDescription]
[WARN] /mnt/c/Users/Esran/Documents/NetbeansProjects/storm/storm-client/src/jvm/org/apache/storm/assignments/InMemoryAssignmentBackend.java:27: <p> tag should be preceded with an empty line. [JavadocParagraph]
[WARN] /mnt/c/Users/Esran/Documents/NetbeansProjects/storm/storm-client/src/jvm/org/apache/storm/assignments/LocalAssignmentsBackendFactory.java:26:5: Missing a Javadoc comment. [JavadocMethod]
[WARN] /mnt/c/Users/Esran/Documents/NetbeansProjects/storm/storm-client/src/jvm/org/apache/storm/assignments/LocalAssignmentsBackendFactory.java:38:5: Missing a Javadoc comment. [JavadocMethod]
[WARN] /mnt/c/Users/Esran/Documents/NetbeansProjects/storm/storm-client/src/jvm/org/apache/storm/blobstore/BlobStore.java:42: Empty line should be followed by <p> tag on the next line. [JavadocParagraph]
[WARN] /mnt/c/Users/Esran/Documents/NetbeansProjects/storm/storm-client/src/jvm/org/apache/storm/blobstore/BlobStore.java:45: Empty line should be followed by <p> tag on the next line. [JavadocParagraph]
[WARN] /mnt/c/Users/Esran/Documents/NetbeansProjects/storm/storm-client/src/jvm/org/apache/storm/blobstore/BlobStore.java:47: Empty line should be followed by <p> tag on the next line. [JavadocParagraph]
{code}

> Upgrade Checkstyle plugin to version 3.0.0
> ------------------------------------------
>
>                 Key: STORM-2891
>                 URL: https://issues.apache.org/jira/browse/STORM-2891
>             Project: Apache Storm
>          Issue Type: Improvement
>    Affects Versions: 2.0.0
>            Reporter: Stig Rohde Døssing
>            Priority: Minor
>              Labels: newbie
>
> It would be good to upgrade the maven-checkstyle-plugin to version 3.0.0. Some pretty nice quality of life improvements there https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12317223&version=12333072.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)