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...@apache.org> on 2015/05/01 19:55:06 UTC
[jira] [Closed] (MCHECKSTYLE-284) Remove config/sun_checks.xml and
use the one built into Checkstyle 6.2+
[ https://issues.apache.org/jira/browse/MCHECKSTYLE-284?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Dennis Lundberg closed MCHECKSTYLE-284.
---------------------------------------
Resolution: Fixed
Assignee: Dennis Lundberg
Fixed in [r1677195|http://svn.apache.org/viewvc?view=revision&revision=1677195].
> Remove config/sun_checks.xml and use the one built into Checkstyle 6.2+
> -----------------------------------------------------------------------
>
> Key: MCHECKSTYLE-284
> URL: https://issues.apache.org/jira/browse/MCHECKSTYLE-284
> Project: Maven Checkstyle Plugin
> Issue Type: Improvement
> Components: predefined ruleset: Sun
> Reporter: Dennis Lundberg
> Assignee: Dennis Lundberg
> Fix For: 2.16
>
>
> Hervé wrote this in MCHECKSTYLE-268:
> {quote}
> Checkstyle 6.2 now has some configs inside Checkstyle jars: see https://github.com/checkstyle/checkstyle/issues/305 and configs that are in resources directory now https://github.com/checkstyle/checkstyle/tree/master/src/main/resources
> With Checkstyle 6.2, <configLocation>sun_checks.xml</configLocation> and <configLocation>google_checks.xml</configLocation> will respectively use Sun and Google coding conventions as included in Checkstyle (the location is a resource name, loaded from classpath)
> I'd like to remove config/sun_checks.xml from maven-checkstyle-plugin source tree, and let user use sun_checks.xml from Checkstyle jar: of course, any other config provided by Checkstyle will be available
> but we'll require Checkstyle 6.2: then we'll need to wait for a later plugin release...
> {quote}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)