Skip to content

Bump spotless to 8.4.0 and ktlint to 1.8.0#52

Merged
ogolberg merged 1 commit intomainfrom
bump-spotless-8
May 8, 2026
Merged

Bump spotless to 8.4.0 and ktlint to 1.8.0#52
ogolberg merged 1 commit intomainfrom
bump-spotless-8

Conversation

@ogolberg
Copy link
Copy Markdown
Member

@ogolberg ogolberg commented May 8, 2026

Summary

  • Bump spotless-plugin-gradle 6.21.0 → 8.4.0 and ktlint 0.50.0 → 1.8.0
  • Raise buildSrc to Java 17 (required by Spotless 8's classpath); consumer plugin artifacts continue to target Java 11
  • Accommodate stricter ktlint defaults: replace a wildcard import, configure ktlint_function_naming_ignore_when_annotated_with for the custom ParameterizedWithGradleVersions test annotation
  • Run spotlessApply across the codebase

Test plan

  • ./gradlew spotlessApply — clean
  • ./gradlew build — compiles; produced .class files are major version 55 (Java 11)
  • Pre-existing FlushJacocoPluginIntegrationTest failure also reproduces on main without these changes, so it is unrelated

Generated with Claude Code

Spotless 8 requires Java 17 bytecode on the buildscript classpath, so
buildSrc is raised to Java 17. Consumer-facing plugin modules continue
to target Java 11.

Spotless 8.1+ dropped support for ktlint < 1.0, so ktlint is bumped to
1.8.0. The stricter defaults required two adjustments:
- replace a wildcard import in TestProjectExtension
- configure ktlint_function_naming_ignore_when_annotated_with for the
  ParameterizedWithGradleVersions annotation, alongside @test

Also runs spotlessApply against the codebase.

Generated with Claude Code
@ogolberg ogolberg merged commit d774eec into main May 8, 2026
1 check passed
@ogolberg ogolberg deleted the bump-spotless-8 branch May 8, 2026 16:57
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