[
https://jira.codehaus.org/browse/MCHECKSTYLE-268?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=361016#comment-361016
]
Herve Boutemy edited comment on MCHECKSTYLE-268 at 1/11/15 2:57 PM:
--------------------------------------------------------------------
until now, Checkstyle configs from Checkstyle are not inside Checkstyle jars
then maven-checkstyle-plugin has its own {{sun_checks.xml}} configuration:
http://svn.apache.org/viewvc/maven/plugins/tags/maven-checkstyle-plugin-2.13/src/main/resources/config/
Checkstyle 6.2 now has its config inside Checkstyle jars: see
https://github.com/checkstyle/checkstyle/issues/305 and configs are in another
source 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...
was (Author: hboutemy):
until now, Checkstyle configs from Checkstyle are not inside Checkstyle jars
then maven-checkstyle-plugin has its own {{sun_checks.xml}} configuration:
http://svn.apache.org/viewvc/maven/plugins/tags/maven-checkstyle-plugin-2.13/src/main/resources/config/
Checkstyle 6.2 now has its config inside Checkstyle jars: see
https://github.com/checkstyle/checkstyle/issues/305 and configs are in another
source directory now
https://github.com/checkstyle/checkstyle/tree/master/src/main/resources
I'd like to remove {{sun_checks.xml}} from maven-checkstyle-plugin source tree,
and get it from Checkstyle jar: of course, any other config provided by
Checkstyle would be supported
but we'll require Checkstyle 6.2: then we'll need to wait for a later plugin
release...
> Add flag/option to use built-in Google style
> --------------------------------------------
>
> Key: MCHECKSTYLE-268
> URL: https://jira.codehaus.org/browse/MCHECKSTYLE-268
> Project: Maven Checkstyle Plugin
> Issue Type: Wish
> Reporter: Christopher Tubbs
>
> Checkstyle 6 adds a style which implements the [Google Style
> Guide|https://code.google.com/p/google-styleguide/], called
> [google_chekcs.xml|https://github.com/checkstyle/checkstyle/blob/master/google_checks.xml].
> It would be nice to expose this style with a flag, providing it with the
> plugin, rather than requiring a user to specify the style file directly. This
> would make it easier for multi-module projects to easily adopt and configure
> a standard style, provided by the upstream Checkstyle project.
--
This message was sent by Atlassian JIRA
(v6.1.6#6162)
