Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make outputSpotbugsFile configurable #96

Closed
mkozioro opened this issue Dec 4, 2018 · 3 comments · Fixed by #130
Closed

Make outputSpotbugsFile configurable #96

mkozioro opened this issue Dec 4, 2018 · 3 comments · Fixed by #130

Comments

@mkozioro
Copy link

mkozioro commented Dec 4, 2018

Hi,

Would it be possible to make outputSpotbugsFileName configurable? I've got couple of old, annoying and unmaintained tool at my company, which is required by our management to collect report from findbugs. It has hardcoded findbugs output file name as findbugsXml.xml. We would like to switch to spotbugs, but I'm not even going to force my company to force 3rd party company to fix their tooling and add support for spotbugs.

Could you consider making output report name configurable? (instead of spotbugsXml.xml). I would be happy to provide PR (adding parameter spotbugsXmlOutputFileName or outputSpotbugsFileName) for that.

Best regards,
Michal

@hazendaz
Copy link
Member

@mkozioro Sorry for delay...if you raise a pull request for this freature, I'll merge it in. The more people using spotbugs the better as findbugs on all fronts has been abandoned at this point and you are otherwise missing out on all the greatness of jdk8+ support ;)

@massdosage
Copy link

We're also interested in this feature. We have a Jenkins installation that has the FindBugs Jenkins plugin for displaying reports. Unfortunately this Jenkins doesn't have the Warnings plugin that supports SpotBugs so if we want to move to SpotBugs but still be able to generate reports in Jenkins we need to be able to modify the output filename to be findbugsXml.xml. We're hoping the XML format is still compatible enough that the older Jenkins plugin can render it. The longer term plan is to move to a newer Jenkins version which has the Warnings plugin but this would be really useful in the mean time. We'll take a look at raising a PR, I had a quick look at the code and it looks relatively straightforward.

@cmathiesen
Copy link

I've raised the PR ^ that hopefully fixes this issue, any feedback or comments is appreciated :)

hazendaz pushed a commit that referenced this issue Jul 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants