Skip to content

Commit

Permalink
Suppress existing spotbugs warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
MarkEWaite authored and hypery2k committed Dec 23, 2023
1 parent 472834d commit 0556d20
Show file tree
Hide file tree
Showing 2 changed files with 34 additions and 27 deletions.
2 changes: 0 additions & 2 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,6 @@
<description>Parses the console log generated by a build</description>

<properties>
<!-- TODO: remove once FindBugs issues are fixed -->
<spotbugs.failOnError>false</spotbugs.failOnError>
<jenkins.version>2.387.3</jenkins.version>
<!-- Maven Plugins -->
<maven-release-plugin.version>2.5.1</maven-release-plugin.version>
Expand Down
59 changes: 34 additions & 25 deletions src/spotbugs/excludesFilter.xml
Original file line number Diff line number Diff line change
@@ -1,29 +1,38 @@
<!--
~ The MIT License
~
~ Copyright (c) 2015, CloudBees, Inc.
~
~ Permission is hereby granted, free of charge, to any person obtaining a copy
~ of this software and associated documentation files (the "Software"), to deal
~ in the Software without restriction, including without limitation the rights
~ to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
~ copies of the Software, and to permit persons to whom the Software is
~ furnished to do so, subject to the following conditions:
~
~ The above copyright notice and this permission notice shall be included in
~ all copies or substantial portions of the Software.
~
~ THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
~ IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
~ FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
~ AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
~ LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
~ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
~ THE SOFTWARE.
-->

<FindBugsFilter>
<!--
Exclusions in this section have been triaged and determined to be
false positives.
-->
<Match>
<Bug pattern="MS_MUTABLE_COLLECTION_PKGPROTECT" />
<Class name="hudson.plugins.logparser.LogParserConsts" />
<Or>
<Field name="LEGAL_STATUS" />
<Field name="STATUSES_WITH_SECTIONS_IN_LINK_FILES" />
</Or>
</Match>

<!--
Here lies technical debt. Exclusions in this section have not yet
been triaged. When working on this section, pick an exclusion to
triage, then:
- Add a @SuppressFBWarnings(value = "[...]", justification = "[...]")
annotation if it is a false positive. Indicate the reason why
it is a false positive, then remove the exclusion from this
section.
- If it is not a false positive, fix the bug, then remove the
exclusion from this section.
-->
<Match>
<Class name="~.*\.Messages"/>
<Bug pattern="DM_DEFAULT_ENCODING" />
<Or>
<Class name="hudson.plugins.logparser.LogParserParser" />
<Class name="hudson.plugins.logparser.LogParserResult" />
<Class name="hudson.plugins.logparser.LogParserUtils" />
<Class name="hudson.plugins.logparser.LogParserWriter" />
<Class name="hudson.plugins.logparser.ReadWriteTextFile" />
</Or>
</Match>
</FindBugsFilter>

0 comments on commit 0556d20

Please sign in to comment.