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

Preview mode for new/enhanced detectors #2946

Open
jrivard opened this issue Apr 14, 2024 · 2 comments
Open

Preview mode for new/enhanced detectors #2946

jrivard opened this issue Apr 14, 2024 · 2 comments

Comments

@jrivard
Copy link

jrivard commented Apr 14, 2024

In several years of using this great tool, I periodically run into issues where a new detector is added or enhanced, but is overly aggressive. In subsequent updates the detector is further refined to be less aggressive / more correct.

Following this cycle can be a bit of a rollercoaster for spotbugs users, one is forced to either rollback the spotbugs version and wait (and in the scope of large projects this can be confusing or forgotten about) or add an exclusion via config or annotation that won't be necessary in the future, and is probably never undone and worst of all - mask actual valid issues that spotbugs has rightfully detected.

To avoid this, would it be possible to add some type of "preview" mode so that new detectors can be tried by the community before they are fully adopted? Ideally this would let new detectors be more aggressive in experimentation and also give more confidence when detectors are finally "promoted" to complete status. Spotbugs users would get warnings (but not errors) about preview detectors - and could also configure preview detectors to cause build errors for testing/evaluation purposes.

I recognize this would add additional overhead to spotbugs development cycles, and only a small subset of spotbugs users would notice/act upon preview detectors - but I still think it would be a net benefit.

Thoughts?

Copy link

welcome bot commented Apr 14, 2024

Thanks for opening your first issue here! 😃
Please check our contributing guideline. Especially when you report a problem, make sure you share a Minimal, Complete, and Verifiable example to reproduce it in this issue.

@ThrawnCA
Copy link
Contributor

Might be better to create new detectors with Experimental priority, rather than have a whole different mode.

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

No branches or pull requests

2 participants