Skip to content

build(deps): Bump build dependencies and sync checkstyle rules with RSyntaxTextArea - #42

Merged
bobbylight merged 1 commit into
masterfrom
bump-build-dependencies-and-checkstyle-rules
Oct 5, 2026
Merged

bobbylight merged 1 commit into
masterfrom
bump-build-dependencies-and-checkstyle-rules

Conversation

@bobbylight

Copy link
Copy Markdown
Owner

Summary

Brings the build tooling and checkstyle rules in line with RSyntaxTextArea 4.x.

Details

Dependency bumps (matching RSyntaxTextArea):

  • SpotBugs Gradle plugin 6.4.8 → 6.5.9
  • Checkstyle 12.3.0 → 13.8.0
  • JUnit BOM 6.0.1 → 6.1.1
  • Mockito 5.21.0 → 5.23.0

Gradle is already at 9.8.0, newer than RSyntaxTextArea's 9.6.1, so it is unchanged.

Checkstyle rules added to config/checkstyle/checkstyle.xml, copied from RSTA changes since 4.0.0:

  • LineEnding
  • IllegalSymbol
  • OverloadMethodsDeclarationOrder (ordered by increasing parameter count)
  • UnusedTryResourceShouldBeUnnamed
  • NumericalPrefixesInfixesSuffixesCharacterCase
  • UseEnhancedSwitch is added commented out, as in RSTA

LineEnding requires LF, so 11 Java source files that had CRLF line endings were converted to LF. That accounts for nearly all of the diff size; use git diff -w --ignore-space-at-eol to review. Nothing else in the sources changed.

Not included: RSTA's javaHeader.txt simplification, which required rewriting the license header in every source file, and the suppressions file, which has no changes in RSTA since 4.0.0.

Test plan

./gradlew clean build passes locally, including the new checkstyle rules.

🤖 Generated with Claude Code

@bobbylight bobbylight added ci/cd dependencies Pull requests that update a dependency file labels Oct 4, 2026
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 82.97632% with 151 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.85%. Comparing base (6c7e624) to head (b2856a8).

Files with missing lines Patch % Lines
...main/java/org/fife/rsta/ui/search/FindToolBar.java 82.72% 45 Missing and 7 partials ⚠️
...fife/rsta/ui/search/AbstractFindReplaceDialog.java 78.00% 22 Missing and 11 partials ⚠️
.../main/java/org/fife/rsta/ui/search/FindDialog.java 68.13% 26 Missing and 3 partials ⚠️
...AUI/src/main/java/org/fife/rsta/ui/GoToDialog.java 78.44% 17 Missing and 8 partials ⚠️
...n/java/org/fife/rsta/ui/search/ReplaceToolBar.java 92.74% 6 Missing and 3 partials ⚠️
...rsta/ui/search/FindReplaceButtonsEnableResult.java 75.00% 2 Missing ⚠️
...ain/java/org/fife/rsta/ui/DecorativeIconPanel.java 96.87% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             master      #42   +/-   ##
=========================================
  Coverage     80.85%   80.85%           
  Complexity      357      357           
=========================================
  Files            22       22           
  Lines          1964     1964           
  Branches        193      193           
=========================================
  Hits           1588     1588           
  Misses          298      298           
  Partials         78       78           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

…SyntaxTextArea

Bumps spotbugs plugin to 6.5.9, checkstyle to 13.8.0, junit-bom to 6.1.1 and
mockito to 5.23.0. Adds the LineEnding, IllegalSymbol,
OverloadMethodsDeclarationOrder, UnusedTryResourceShouldBeUnnamed and
NumericalPrefixesInfixesSuffixesCharacterCase checkstyle rules, and converts
Java sources with CRLF line endings to LF to satisfy LineEnding.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@bobbylight
bobbylight force-pushed the bump-build-dependencies-and-checkstyle-rules branch from b2856a8 to 0d9bf48 Compare October 5, 2026 00:47
@bobbylight bobbylight self-assigned this Oct 5, 2026
@bobbylight bobbylight added this to the 4.0.0 milestone Oct 5, 2026
@bobbylight
bobbylight marked this pull request as ready for review October 5, 2026 00:48
@bobbylight
bobbylight merged commit a3d8cbf into master Oct 5, 2026
3 checks passed
@bobbylight
bobbylight deleted the bump-build-dependencies-and-checkstyle-rules branch October 5, 2026 00:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants