Cargo
  1. Cargo
  2. CARGO-921

Apply checkstyle on the cargo-maven2-plugin

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.1.0
    • Component/s: Maven2
    • Labels:
      None
    • Complexity:
      Intermediate
    • Number of attachments :
      0

      Description

      For some reason, checkstyle is disabled for the cargo-maven2-plugin. Enable and correct.

        Issue Links

          Activity

          Savas Ali Tokmen made changes -
          Field Original Value New Value
          Fix Version/s 1.1.0 [ 16436 ]
          Savas Ali Tokmen made changes -
          Link This issue is depended upon by CARGO-920 [ CARGO-920 ]
          Savas Ali Tokmen made changes -
          Link This issue relates to CARGO-891 [ CARGO-891 ]
          Hide
          Ate Douma added a comment -

          Current checkstyle plugin 2.6 (checkstyle 5.0) reports errors for Maven Mojo annotations.
          This seemingly is fixed with checkstyle 5.1+ and maven-checkstyle-plugin 2.7, see: MCHECKSTYLE-132
          However, that version hasn't been released yet.

          Show
          Ate Douma added a comment - Current checkstyle plugin 2.6 (checkstyle 5.0) reports errors for Maven Mojo annotations. This seemingly is fixed with checkstyle 5.1+ and maven-checkstyle-plugin 2.7, see: MCHECKSTYLE-132 However, that version hasn't been released yet.
          Hide
          Savas Ali Tokmen added a comment -

          Committed revision 2750.

          Show
          Savas Ali Tokmen added a comment - Committed revision 2750.
          Savas Ali Tokmen made changes -
          Status Open [ 1 ] Closed [ 6 ]
          Assignee Savas Ali Tokmen [ alitokmen ]
          Resolution Fixed [ 1 ]

            People

            • Assignee:
              Savas Ali Tokmen
              Reporter:
              Savas Ali Tokmen
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: