Uploaded image for project: 'Flume'
  1. Flume
  2. FLUME-2937

Integrate checkstyle for non-test classes

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

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.7.0
    • Component/s: None
    • Labels:
      None

      Description

      We should add the maven-checkstyle-plugin to the build process. This plugin can fail a build if the code does not honor the style of our project. This way we can make sure that we have one common style in the code. In addition, reviewers can focus on design, correctness, performance and other important coding aspects other than style issues.

        Attachments

        Issue Links

          Activity

            People

            • Assignee:
              mpercy Mike Percy
              Reporter:
              liorze Lior Zeno

              Dates

              • Created:
                Updated:
                Resolved:

                Issue deployment