[jira] [Commented] (MCHECKSTYLE-314) checkstyle:check should invoke the execution of this plugin's goal "checkstyle" prior to executing itself

["Michael Osipov (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/MCHECKSTYLE-314?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15045466#comment-15045466 ] 

Michael Osipov commented on MCHECKSTYLE-314:
--------------------------------------------

This is merely {{@Execute( goal = "checkstyle" )}} in {{CheckstyleViolationCheckMojo}}. If no dev opposes to that, I would go ahead and commit this change.

> checkstyle:check should invoke the execution of this plugin's goal "checkstyle" prior to executing itself
> ---------------------------------------------------------------------------------------------------------
>
>                 Key: MCHECKSTYLE-314
>                 URL: https://issues.apache.org/jira/browse/MCHECKSTYLE-314
>             Project: Maven Checkstyle Plugin
>          Issue Type: Improvement
>    Affects Versions: 2.17
>            Reporter: Roman Ivanov
>            Assignee: Michael Osipov
>
> I run into problem with using checkstyles goal "checkstyle:check" in sevntu.checkstyle project (https://github.com/sevntu-checkstyle/sevntu.checkstyle/blob/master/sevntu-checks/pom.xml#L75) as it always use default Sun config when I run "checkstyle:check", It took me a while to figure out why that does not work.
> I had a habit and convenience to run in such a ways pmd and findbug, just "mvn pmd:check", "mvn findbug:check" it is human friendly.
> PMD and Findbugs plugins already do this:
> http://gleclaire.github.io/findbugs-maven-plugin/check-mojo.html
> https://maven.apache.org/plugins/maven-pmd-plugin/check-mojo.html
> search for "Invokes the execution of this plugin's goal"
> vs
> http://maven.apache.org/plugins/maven-checkstyle-plugin/check-mojo.html
> Problem was also raised at : http://stackoverflow.com/questions/11106767/maven-checkstylecheck-not-working
> Please add support of this, most users are not professionals in Maven. I use Maven for many years and it took me too much time to find a reason why it does not work.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)