spotbugs-gradle-plugin
spotbugs-gradle-plugin copied to clipboard
spotbugs
Maybe consider renaming spotbugsMain to spotbugs?
For most standard Gradle plugins, the Main is left out for the main source set in task names, like compileJava vs. compileTestJava or compileOtherSourceSetJava and so on.
Maybe it would make sense to follow suit and also name the SpotBugs task for the main source set just spotbugs?
Just an idea, if you don't like it, just close it. :-)
Really interesting, if we can handle backward compatibility well, it should be nice. :)
Well, you could rename the task to spotbugs, then add a simple no-op task called spotbugsMain that depends on spotbugs and emits a deprecation warning, then at some point remove it.
Alternatively to catch the guys that read no warnings and / or have it in some scripts as an intermediary step between deprecation warning and removal make the task fail the build so that a meaningful error message can be given and then later on remove it.
Hm, actually I'm unsure now. The standard code quality plugins shipped with Gradle like codenarc, pmd, findbugs or jdepend all follow the ´blaMain` convention. :-/ I'll open a ticket to Gradle first to see what they think about it. I'll probably tend to staying in-line with the standard CQ plugins if they don't change it.
It seems that the discussion at that issue is still in progress, so I removed milestone from this issue.