Skip to content

JAVA-126: Use .sdkmanrc as Java version source for CI#31

Merged
lukaszgyg merged 1 commit intomainfrom
feature/use-sdkmanrc-for-ci-java-version
Feb 11, 2026
Merged

JAVA-126: Use .sdkmanrc as Java version source for CI#31
lukaszgyg merged 1 commit intomainfrom
feature/use-sdkmanrc-for-ci-java-version

Conversation

@lukaszgyg
Copy link
Copy Markdown
Contributor

Summary

This PR migrates the repository to use .sdkmanrc as the single source of truth for Java version in CI workflows.

Changes

  • Created .sdkmanrc file with java=17.0.13-tem
  • Updated actions/setup-java from v3.13.0/v4.4.0 to v5.1.0 in all workflows
  • Changed java-version: 17 parameter to java-version-file: .sdkmanrc in:
    • .github/workflows/codeql.yml
    • .github/workflows/dependency-submission.yml

Benefits

  • Single source of truth for Java version across CI and local development
  • Improved consistency between CI and developer environments
  • Easier to maintain and update Java versions

Migration Guide

Follows: agent-migrations/2026-01-sdkmanrc-ci-java-version-technical-guide.md

Testing

  • All existing workflow files have been updated
  • CI will use Java 17.0.13 (Temurin distribution) from .sdkmanrc

- Created .sdkmanrc file
- Updated setup-java action to v5.1.0
- Changed java-version parameter to java-version-file: .sdkmanrc

Follows: agent-migrations/2026-01-sdkmanrc-ci-java-version-technical-guide.md

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@lukaszgyg lukaszgyg requested a review from a team as a code owner January 24, 2026 10:39
@lukaszgyg
Copy link
Copy Markdown
Contributor Author

lukaszgyg commented Jan 30, 2026

👋 Hi! This is a friendly automated reminder about this PR.

What this PR does:
This PR migrates the CI workflow to use .sdkmanrc as the single source of truth for the Java version, instead of hardcoding it in GitHub Actions workflows.

Why this is safe:

  • ✅ All CI checks are passing
  • ✅ No code changes, only workflow configuration
  • ✅ The Java version stays the same, just read from a different source
  • ✅ Improves consistency between local dev and CI environments

What's needed:
Just a quick review and approval when you have a moment! 🙏

More info:


This is an automated message for paparazzi-plugin #31. If you have questions, feel free to comment!

@lukaszgyg lukaszgyg merged commit 647641c into main Feb 11, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant