Upgrade to Spotbugs 6.0.15 #1398
Conversation
* Exclude CT_CONSTRUCTOR_THROW * Exclude PA_PUBLIC_PRIMITIVE_ATTRIBUTE * Add assertion to handle NP warning JAVA-5480
config/spotbugs/exclude.xml
Outdated
| @@ -18,6 +18,17 @@ | |||
| for the mapping from SpotBugs rank to verbal labels. | |||
| --> | |||
| <FindBugsFilter> | |||
| <Match> | |||
| <!-- MongoDB status: "No Fix Needed", SpotBugs rank: * --> | |||
There was a problem hiding this comment.
@stIncMale how did you determine the ranks? Above comment says to look at https://spotbugs.readthedocs.io/en/latest/filter.html#rank but I don't see anything useful at that location.
There was a problem hiding this comment.
I determined the ranks as described in #1392:
To get the rank data, I had enabled SpotBugs XML reports, then was commenting out parts of exclude.xml and manually get the rank data from the XML reports.
There was a problem hiding this comment.
Above comment says to look at https://spotbugs.readthedocs.io/en/latest/filter.html#rank but I don't see anything useful at that location.
It says to look at the page "for the mapping from SpotBugs rank to verbal labels", and the mapping is there. This is in case we decide to map the numbers to something more human-friendly.
There was a problem hiding this comment.
Ah, got it. I added the ranks and a comment saying how to find them, for our future selves.
JAVA-5480