Skip to content

build(deps): bump checkstyle, JUnit, Mockito and Rhino - #118

Merged
bobbylight merged 1 commit into
masterfrom
feature/bump-dependencies
Oct 16, 2026
Merged

bobbylight merged 1 commit into
masterfrom
feature/bump-dependencies

Conversation

@bobbylight

@bobbylight bobbylight commented Oct 15, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • checkstyle tool version 12.3.0 -> 14.3.0
  • junit-bom 6.0.1 -> 6.1.3
  • mockito-core 5.21.0 -> 5.24.0
  • rhino-all 1.9.0 -> 1.9.1
  • Enables new checkstyle rules: SummaryJavadoc, ArrayBracketNoWhitespace, UnnecessaryTypeArgumentsWithRecordPattern, MultilineCommentLeadingAsteriskPresence; removes JavadocStyle (no longer exists in newer checkstyle)
  • Fixes the violations the new rules surfaced: ~390 Javadoc comments missing a first-sentence period or a summary (@return-only Javadoc gets a "Returns ..." summary), and leading asterisks added to multi-line comments (mostly commented-out code)

Already at latest: Gradle wrapper (9.8.1), RSyntaxTextArea (4.0.1), AutoComplete (4.0.0), the nexus publish plugin (2.0.0). GitHub Actions versions are left to Dependabot.

The bulk of the diff is Javadoc/comment-only; no code logic changes.

Test plan

  • ./gradlew clean build -xsign passes (compile, checkstyle, tests) on JDK 25
  • CI green on JDK 25 and 26

🤖 Generated with Claude Code

@bobbylight bobbylight added the dependencies Pull requests that update a dependency file label Oct 15, 2026
@bobbylight bobbylight self-assigned this Oct 15, 2026
@bobbylight bobbylight added this to the 4.0.0 milestone Oct 15, 2026
…eckstyle rules

- checkstyle tool version 12.3.0 -> 14.3.0
- junit-bom 6.0.1 -> 6.1.3
- mockito-core 5.21.0 -> 5.24.0
- rhino-all 1.9.0 -> 1.9.1
- Enables new checkstyle rules: SummaryJavadoc, ArrayBracketNoWhitespace,
  UnnecessaryTypeArgumentsWithRecordPattern, MultilineCommentLeadingAsteriskPresence;
  removes JavadocStyle (no longer exists in newer checkstyle)
- Fixes the violations the new rules surfaced (mostly missing Javadoc summaries
  and first-sentence periods, plus leading asterisks in commented-out code)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@bobbylight
bobbylight force-pushed the feature/bump-dependencies branch from 8119201 to 7a116df Compare October 15, 2026 23:12
@bobbylight
bobbylight marked this pull request as ready for review October 16, 2026 01:17
@bobbylight
bobbylight merged commit abc99c4 into master Oct 16, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant