Future of this plugin?

[tomasbjerre]

I think this plugin needs a new maintainer. If you want to be one, you can ask for commit access in Jenkins developer mailing list:
https://wiki.jenkins-ci.org/display/JENKINS/Adopt+a+Plugin

I may be interested in being a maintainer. But if so, I would make very big changes in it. My opinion is that the code is just to messy to fix. I will not spend more time fixing things like this, and there is alot of that here.

I would suggest that:

• A library for parsing static code analasys reports is first developed. I've started here.
• This plugin should use analysis-core to present the results.

And replace the current code with that implementation. That would make this plugin much less complex and maintainable.

What do you think?
@ryoichitaniguchi @rsommerard @MensObscura @nbruno @pfhorman @tandibar @wang-xinhong

[pfhorman]

I agree that the code isn't the cleanest and can be tricky to understand, and that analysis-core probably should be used if starting over. In the meantime, I think there are some smaller changes queued up as pull requests that could benefit the current users sooner rather than later (ie. waiting for a re-write).

[wang-xinhong]

Quite a nice idea to separate such analyzing things from user interface. But I found it's not clear that whether the dependence - analysis-core would go steady within the foreseeable future.

[gliptak]

Maybe a number of the pull requests could be committed first? Thanks

[tuukkamustonen]

+1. The last stable version (that is installable via Plugin manager) is 0.7.11 from Sep 8, 2012, which kinda suggests one to avoid the plugin altogether.

Static Code Analysis plugins look awesome. As there's no development going on here anyway, just start fresh...

[gliptak]

A number of file formats supported by Violations Plugin is not supported by the Static Code Analysis based plugins ...

[tuukkamustonen]

@gliptak I'm not Jenkins expert so maybe I misunderstood something, but wouldn't the idea be to use violations-lib for analyzing the output files and then "somehow" to integrate into analysis-core and analysis-collector for displaying those results?

[gliptak]

Yes a significant rewrite would allow being based on analysis-core.

[tomasbjerre]

I started fiddling with this here: https://github.com/jenkinsci/violations-plugin/tree/violations-lib

[tomasbjerre]

https://groups.google.com/forum/#!topic/jenkinsci-dev/KjyIOtg0wWs

[tomasbjerre]

I got it working with the lib and analysis core now.

If the switch is made now, that means these formats are no longer supported (I will implemented as soon as I get my hands on example reports and have the time):

• Codenarc
• Cpd
• Gendarme
• Jcereport
• Simian
• ZptLint

And these are added:

• AndroidLint
• CppCheck
• Flake8
• PiText

[gliptak]

A number of sample reports are checked into the tree https://github.com/jenkinsci/violations-plugin/tree/master/src/test/resources/hudson/plugins/violations/types

[tomasbjerre]

Nice, they are all implemented now.

[gliptak]

@tomasbjerre that was quick 👍

[tomasbjerre]

I'll make a 1.0 release as soon as I get accepted as maintainer.