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

Replace Findbugs with Spotbugs? #42

Open
lksnmnn opened this issue Jun 19, 2019 · 2 comments
Open

Replace Findbugs with Spotbugs? #42

lksnmnn opened this issue Jun 19, 2019 · 2 comments

Comments

@lksnmnn
Copy link

lksnmnn commented Jun 19, 2019

Hi there,

one of the projects I inherited is using android-check. I recently updated all outdated dependencies and found that Findbugs will not be supported in future Gradle versions

Since FindBugs is unmaintained and does not support bytecode compiled for Java 9 and above, the FindBugs plugin has been deprecated and is scheduled to be removed in Gradle 6.0. Please consider using the SpotBugs plugin instead.

Source: https://docs.gradle.org/current/userguide/findbugs_plugin.html

Are there any plans moving from Findbugs to Spotbugs in the future? Maybe we can throw together a PR for this?

Cheers,
Lukas

@FriedrichFroebel
Copy link

This change should be relatively easy and straightforward. When testing this some time ago using SpotBugs 3.1.0, I got some classpath length errors with a project which worked fine using FindBugs before - I do not know whether this has been an actual bug with SpotBugs and may be fixed with the current version. If changed, the names of the configuration closures should be evaluated as well - whether to use spotbugs instead of findbugs or whether not to break compatibility with existing setups by using the findbugs name.

The android-check plugin repository does not seem to receive any maintenance since the end of 2017. My PR to update the other library versions to the latest versions (from when I created the PR) is still awaiting response from the maintainers after about a year for example.

@lksnmnn
Copy link
Author

lksnmnn commented Jun 20, 2019

The android-check plugin repository does not seem to receive any maintenance since the end of 2017.

That is too bad, but was my first impression too. I removed android-check all together and added the 3 plugins manually. I took me some initial work as I am not familiar with the Java / Android / Gradle toolchain. But if @stefan-nsk or @smalichenko decide to keep mainting this plugin I am open to contribute back :)

Also, if you guys drop this project you should mark it as discontinued / unmaintained so ppl dont use this in their projects any longer.

Cheers,
Lukas

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