[ 
https://jira.codehaus.org/browse/MCHECKSTYLE-252?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Herve Boutemy updated MCHECKSTYLE-252:
--------------------------------------

    Description: 
Updating maven-checkstyle-plugin to version 2.13 and running

{noformat}$ mvn site{noformat}

now includes non-source files in its checks, e.g.

{noformat}COPYING
COPYING.LESSER
target/generated-classes/cobertura/cobertura.properties{noformat}

While it is rather amusing to have checkstyle complain that COPYING (the text 
of the GNU General Public License) doesn't include a license header, it seems 
that the default source root or one of the other properties new to version 2.13 
does not have an appropriate default value.  The target directory should also 
be excluded by default.

  was:
Updating maven-checkstyle-plugin to version 2.13 and running

$ mvn site

now includes non-source files in its checks, e.g.

COPYING
COPYING.LESSER
target/generated-classes/cobertura/cobertura.properties

While it is rather amusing to have checkstyle complain that COPYING (the text 
of the GNU General Public License) doesn't include a license header, it seems 
that the default source root or one of the other properties new to version 2.13 
does not have an appropriate default value.  The target directory should also 
be excluded by default.


> Version 2.13 includes non-source files in checks
> ------------------------------------------------
>
>                 Key: MCHECKSTYLE-252
>                 URL: https://jira.codehaus.org/browse/MCHECKSTYLE-252
>             Project: Maven Checkstyle Plugin
>          Issue Type: Bug
>          Components: checkstyle:checkstyle
>    Affects Versions: 2.13
>            Reporter: Michael Heuer
>             Fix For: 2.14
>
>         Attachments: mcheckstyle-252.tar.gz
>
>
> Updating maven-checkstyle-plugin to version 2.13 and running
> {noformat}$ mvn site{noformat}
> now includes non-source files in its checks, e.g.
> {noformat}COPYING
> COPYING.LESSER
> target/generated-classes/cobertura/cobertura.properties{noformat}
> While it is rather amusing to have checkstyle complain that COPYING (the text 
> of the GNU General Public License) doesn't include a license header, it seems 
> that the default source root or one of the other properties new to version 
> 2.13 does not have an appropriate default value.  The target directory should 
> also be excluded by default.



--
This message was sent by Atlassian JIRA
(v6.1.6#6162)

Reply via email to