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

Integrate checkstyle for non-test classes

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • None
    • 1.7.0
    • None
    • 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

          blocks

          Improvement - An improvement or enhancement to an existing feature or task. FLUME-2941 Integrate checkstyle for test classes

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved
          is cloned by

          Improvement - An improvement or enhancement to an existing feature or task. FLUME-2941 Integrate checkstyle for test classes

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved
          relates to

          Improvement - An improvement or enhancement to an existing feature or task. FLUME-2943 Integrate checkstyle - second pass

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Open
          links to

          Web Link Code review

          Activity

            People

              mpercy Mike Percy
              liorze Lior Zeno
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: