Gradle checkstylew default version should be increased

Old Forum Archive
1

Latest checkstyle version is 5.7 as of today: http://mvnrepository.com/artifact/com.puppycrawl.tools/checkstyle

With gradle 2.0, it seems this is still not 5.7 (not sure which version is used instead).

This affects new functionalities like here: http://stackoverflow.com/questions/4023185/how-to-disable-a-particular-checkstyle-rule-for-a-particular-line-of-code

<module name="SuppressWarningsFilter" />
    <module name="TreeWalker">
        <module name="SuppressWarningsHolder" />

Current workaround is to add to build.gradle:

dependencies {
    checkstyle('com.puppycrawl.tools:checkstyle:5.7')
}

or with newer guava:

dependencies {
    checkstyle('com.puppycrawl.tools:checkstyle:5.7') {
        exclude group: 'com.google.guava'
    }
    checkstyle("com.google.guava:guava:17.0") { force = true }
}

If there is any reason not to use latest checkstyle, at least the checkstyle page could provide an example of how to use a later version, such as the snippet above.

2

That’s exactly what the ‘checkstyle’ dependency is for - to be able to use a selected version of checkstyle. It is described in section 29.4 of the manual here: http://www.gradle.org/docs/current/userguide/checkstyle_plugin.html#N133C7

3

Yes, I am aware of that page and that’s how I managed to find the solution, eventually. However, IMHO, the default version should always be the newest (stable). And second, the documentation is understandable for Gradle experts, but most users use the gradle docs just to copy&paste snippets into their build.gradle file, and I think the docs should make that process easier by giving an example.

4

Ideally users should only need to specify a version if they do not want the newest (upgrade opt-out), IMHO.

5

So BTW, default checkstyle version in gradle 1.12 is 5.6, in 2.0 it is 5.7. So it seems the updates are in place anyway.

6

That doesn’t seem to be explained in 29.4 of the doc anymore?