You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Brent Ryan (JIRA)" <ji...@codehaus.org> on 2015/01/08 17:39:18 UTC

[jira] (MCHECKSTYLE-252) Version 2.13 includes non-source files in checks

    [ https://jira.codehaus.org/browse/MCHECKSTYLE-252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=360968#comment-360968 ] 

Brent Ryan commented on MCHECKSTYLE-252:
----------------------------------------

I want to add that this also seems to have caused checkstyle to include target/generated-sources and there doesn't appear to be anyway to exclude it.

This is pretty huge because it means we can't use checkstyle at all for Java8 projects.

> 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
> $ 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.



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