Cargo

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

is depended upon by

Improvement - An improvement or enhancement to an existing feature or task. CARGO-920 Support for Maven2 child modules and fixed/improved shared/extra classpath configurations with Tomcat

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
relates to

Improvement - An improvement or enhancement to an existing feature or task. CARGO-891 Also checkstyle the test Java files

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
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
Permalink
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
Permalink
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
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: