chore: remove spotless check override for Java 25

[curfew-marathon]

Removes the spotless check override for Java 25 in the CI workflow. Previously, Java 25 builds were skipping spotlessCheck and only running build test-integration. This change ensures all Java versions (17, 21, 25) run the full build process consistently, including code formatting checks.

Summary by CodeRabbit

• Chores
  • Simplified the build and test pipeline by consolidating the integration workflow, removing version-specific conditional logic for more consistent and streamlined execution.

[coderabbitai]

Walkthrough

The .github/workflows/main.yaml CI workflow was simplified to unconditionally run ./gradlew build test-integration, removing the Java 25-specific conditional that previously skipped the Spotless check task on that version.

Changes

Cohort / File(s)	Summary
CI Workflow Simplification .github/workflows/main.yaml	Removed Java 25-specific conditional logic that excluded spotlessCheck, replacing it with a single unconditional Gradle command that consistently runs build and integration tests across all Java versions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• chore: Add Java 25 to CI build matrix with conditional Spotless skip #286: Adds Java 25 to the workflow with a conditional skip of Spotless; this PR reverses that conditional logic by always running Spotless regardless of Java version.
• Chore/upgrade spotless to 8.2.1 #285: Upgrades Spotless and modifies the Java matrix in the GitHub Actions workflow; related changes to the same workflow's Gradle/Spotless task handling.
• chore: added 'make build' and fixed gradle warning #284: Adds a Makefile build target with Gradle invocation and adjusts Spotless configuration, affecting how build and Spotless tasks are orchestrated.

Suggested reviewers

• jimmyjames
• ewanharris
• rhamzeh

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the main change: removing a spotless check override for Java 25 in CI workflow, which aligns with the changeset that simplifies the build process by eliminating conditional logic.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection	✅ Passed	✅ No merge conflicts detected when merging into main

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch chore/remove-spotless-override-java25

---

No actionable comments were generated in the recent review. 🎉

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 37.70%. Comparing base (29c11aa) to head (925028e).

Additional details and impacted files

@@            Coverage Diff            @@
##               main     #287   +/-   ##
=========================================
  Coverage     37.70%   37.70%           
  Complexity     1236     1236           
=========================================
  Files           197      197           
  Lines          7609     7609           
  Branches        880      880           
=========================================
  Hits           2869     2869           
  Misses         4601     4601           
  Partials        139      139           

☔ View full report in Codecov by Sentry.
📢 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.

[dosubot]

Related Documentation

Checked 8 published document(s) in 1 knowledge base(s). No updates required.

^{How did I do? Any feedback?}  

[Copilot]

Pull request overview

This PR updates the main CI workflow to run the same Gradle task set across all Java versions in the test matrix, removing the special-casing that skipped spotlessCheck on Java 25.

Changes:

• Removed the Java 25 conditional that excluded spotlessCheck.
• Standardized the CI Gradle invocation to ./gradlew build test-integration for Java 17/21/25.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.