Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.14.0
    • Component/s: None
    • Labels:
      None

      Description

      Upgrade maven-checkstyle-plugin from 2.12.1 (checkstyle 5.7) to 2.17 (checkstyle 6.11.2). It should include bug-fixes and allow some new checks.

      As part of this change, fix any issues. Expect lots of cosmetic changes (especially whitespace).

        Activity

        Show
        julianhyde Julian Hyde added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/calcite/commit/9ab47c00 .
        Hide
        julianhyde Julian Hyde added a comment -

        A further fix in http://git-wip-us.apache.org/repos/asf/calcite/commit/1f04d556. Checkstyle was checking generated parser files, and was generating a lot of violations.

        Show
        julianhyde Julian Hyde added a comment - A further fix in http://git-wip-us.apache.org/repos/asf/calcite/commit/1f04d556 . Checkstyle was checking generated parser files, and was generating a lot of violations.
        Hide
        michaelmior Michael Mior added a comment -

        Resolved in release 1.14.0 (2017-10-01)

        Show
        michaelmior Michael Mior added a comment - Resolved in release 1.14.0 (2017-10-01)

          People

          • Assignee:
            julianhyde Julian Hyde
            Reporter:
            julianhyde Julian Hyde
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development