Skip to content

Add SpotBugs static analysis check to CI pipeline#117

Merged
bernardladenthin merged 1 commit into
mainfrom
claude/friendly-feynman-as1vh1
Jun 26, 2026
Merged

Add SpotBugs static analysis check to CI pipeline#117
bernardladenthin merged 1 commit into
mainfrom
claude/friendly-feynman-as1vh1

Conversation

@bernardladenthin

Copy link
Copy Markdown
Owner

Summary

  • Add SpotBugs static analysis check to the publish workflow to catch potential bugs early in CI
  • Run SpotBugs during the build phase with test compilation and enforcer checks skipped to fail fast on findings
  • Complements existing Spotless formatting checks with additional code quality validation

Test plan

  • CI is green on this branch (SpotBugs check passes with no violations)

Related issues / PRs

Checklist

  • I have read CONTRIBUTING.md and CODE_OF_CONDUCT.md
  • My commits follow Conventional Commits
  • No security-sensitive changes (if there are, I have notified the maintainer privately per SECURITY.md)

https://claude.ai/code/session_0137c1LhUbNvW3kt4eF9Kqyb

spotbugs:check is bound to verify. Add a SpotBugs step to the existing fast
code-style job (after Spotless, before the informational jdeps step) so it
runs early on every PR/push, consistent with the other sibling repos;
publish-* already needs: code-style. Mirrors the existing early Spotless gate.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0137c1LhUbNvW3kt4eF9Kqyb
@bernardladenthin bernardladenthin merged commit 85e950c into main Jun 26, 2026
10 of 11 checks passed
@bernardladenthin bernardladenthin deleted the claude/friendly-feynman-as1vh1 branch June 26, 2026 06:57
@sonarqubecloud

Copy link
Copy Markdown

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.

2 participants