You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Dennis Lundberg (JIRA)" <ji...@codehaus.org> on 2009/07/10 08:49:23 UTC

[jira] Closed: (MCHECKSTYLE-98) Maven Checkstyle is too strict and not follow Maven's team conventions!

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

Dennis Lundberg closed MCHECKSTYLE-98.
--------------------------------------

         Assignee: Dennis Lundberg
       Resolution: Fixed
    Fix Version/s: 2.3

I've applied the changes agreed upon, see my previous comment.

I have not applied Benjamin's patch, since that check only produces warnings. At this point in time my main concern is getting errors reported for the right things.

> Maven Checkstyle is too strict and not follow Maven's team conventions!
> -----------------------------------------------------------------------
>
>                 Key: MCHECKSTYLE-98
>                 URL: http://jira.codehaus.org/browse/MCHECKSTYLE-98
>             Project: Maven 2.x Checkstyle Plugin
>          Issue Type: Task
>    Affects Versions: 2.2
>            Reporter: Vincent Siveton
>            Assignee: Dennis Lundberg
>             Fix For: 2.3
>
>         Attachments: hidden-field.patch, maven_style_improvements.diff, site-MPIR-improvements.zip, site-MPIR-r675999.zip
>
>
> The Checkstyle report is an official part of the Maven's project website. Release manager needs to review it before cutting a release.
> Actually, the Maven Checkstyle conf is too strict and not following our conventions.
> Lets take the MPIR project (rev 675999)  and see site-MPIR-r675999.zip attachment. There is a lot of unuseful info/warn ie "Missing a Javadoc comment" or "Expected @param tag for..."
> I propose to modify the current Maven style (see maven_style_improvements.diff). You could see results in site-MPIR-improvements.zip

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira