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

Use separate artifactId for spotbugs version #313

Closed
kemitix opened this Issue Nov 13, 2018 · 4 comments

Comments

Projects
None yet
3 participants
@kemitix
Copy link

kemitix commented Nov 13, 2018

Dependency management tools regard 7.4.3 to be an upgrade from 7.4.3.sb.

Given that the findbugs and spotbugs 'versions' of fb-contrib are essentially targetting different APIs, they should really have different artifactIds. This would also better align versioning with Semantic Versioning.

@mebigfatguy

This comment has been minimized.

Copy link
Owner

mebigfatguy commented Nov 16, 2018

yeah you're right. i had assumed i'd just drop the findbugs versions after awhile, but then never could figure out when a good time was.

@mebigfatguy

This comment has been minimized.

Copy link
Owner

mebigfatguy commented Dec 30, 2018

if you could review these changes to make sure i didn't fubar something, would be appreciated.

@mebigfatguy

This comment has been minimized.

Copy link
Owner

mebigfatguy commented Dec 30, 2018

(i will have to update README.md and the the other docs once a split release goes out)

@ThrawnCA

This comment has been minimized.

Copy link
Contributor

ThrawnCA commented Dec 30, 2018

On that note, the README refers to an obsolete version of SpotBugs Maven Plugin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment