Skip to content

Clear remaining ignored SpotBugs findings - #41

Merged
e6qu merged 1 commit into
mainfrom
phase-33/spotbugs-baseline-drawdown
May 17, 2026
Merged

Clear remaining ignored SpotBugs findings#41
e6qu merged 1 commit into
mainfrom
phase-33/spotbugs-baseline-drawdown

Conversation

@e6qu

@e6qu e6qu commented May 17, 2026

Copy link
Copy Markdown
Owner

Summary

  • remove the SpotBugs exclude filter so high-confidence findings are no longer ignored
  • make ThemeCache color arrays private behind typed accessors
  • fix the previously hidden vendored NFS/RPC and sun.net.www SpotBugs findings
  • update BUGS/PLAN and SpotBugs workflow docs to reflect the unfiltered setup

Validation

  • git diff --check
  • ./gradlew spotbugsMain spotbugsTest --stacktrace
  • ./gradlew cleanTest test --stacktrace
  • ./gradlew check --stacktrace
  • ./gradlew :barebones-protocol-nfs:compileJava :barebones-protocol-nfs:spotbugsMain --stacktrace

@e6qu
e6qu merged commit 053f7cf into main May 17, 2026
6 checks passed
@e6qu
e6qu deleted the phase-33/spotbugs-baseline-drawdown branch May 17, 2026 10:55
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

Successfully merging this pull request may close these issues.

1 participant