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
Detected NPE causes EP's Var rule to go nuts. #923
Comments
I can't reproduce this. I cloned the repo and checked out the
I was building using JDK 17. |
CI reproduces it, tests are set to require java 21, while compile is 11. https://github.com/xenoterracide/gradle-semver/actions/runs/8066098577/job/22033383615 |
Interesting. I cannot repro locally on a Mac building with JDK 11, 17, or 21. Not sure what's happening. In any case I expect the issue is with the Var check and not with NullAway. |
hmm... I run linux locally, and so does the builder... I wonder if it could secretly have to do with Java on Linux... any chance you have the ability to try that? I sadly do not have access to a Mac. |
Maybe you can spin up a Mac in your CI config to see if there is a difference? Unfortunately I don't have time to try on Linux right now, don't have a box handy. |
maybe this is an issue with errorprone, I'll cross report. None of these
@Var
's are not effectively final.this.git
is variable though.Full source here https://github.com/xenoterracide/gradle-semver/blob/external/bug/ep/buildSrc/src/main/kotlin/our.javacompile.gradle.kts
google/error-prone#4305
The text was updated successfully, but these errors were encountered: