mk868
mk868
There is also `Objects.requireNonNullElse`: ```java var semver = Objects.requireNonNullElse(Semver.coerce(string), Semver.of(0, 0, 0)); var semver = Objects.requireNonNull(Semver.coerce(string); ```
@joerg1985 At the moment we have ~80 warnings. I think setting the ERROR level will cause the compilation to fail I have such steps in my mind: 1. For each...
Agree, So, I'll modify PR: - I'll set NullAway reporting level to `ERROR` - I'll make Bazel optionally run NullAway analysis via flag/tag/argument (I need to check that) - I'll...
/describe
/review
Hi @joerg1985, @diemol I updated the Pull Request. Now nullness checks are part of the workflow (NullAway set to log issues with the error level) - I added `nullaway_level` flag...
Branch rebased
Hello, I just pushed two commits, with changes: - the `trunk' branch was merged - NullAway updated to the `0.11.3` - the code was reformatted I'm still not sure if...
Is there any chance to merge this? Or should I apply some changes / split the PR into several smaller ones?
To be honest, these SpotBugs errors are confusing to me because there is no clear error message From what I see NullAway and Spotbugs, don't have common transitive dependencies which...