Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include setup-scala step in CI #126

Merged
merged 2 commits into from
Jan 2, 2025
Merged

Include setup-scala step in CI #126

merged 2 commits into from
Jan 2, 2025

Conversation

kelvin-chappell
Copy link
Member

This replaces the setup-java step with setup-scala now that SBT has to be installed as well.

Setting Java version to latest Corretto LTS release.

.tool-versions Outdated
@@ -0,0 +1 @@
java corretto-21.0.5.11.1
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We were using Java 11 with actions/setup-java. Here we're using Java 21. Have we tested with this version?

Copy link
Member Author

@kelvin-chappell kelvin-chappell Dec 20, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The deployment process here is a bit mysterious and I don't know what version of Java it uses. I have only tested the CI workflow. I'll set the version to 11 to keep this as an atomic change.

@kelvin-chappell kelvin-chappell merged commit 3608563 into main Jan 2, 2025
1 check passed
@kelvin-chappell kelvin-chappell deleted the kc/gha-scala branch January 2, 2025 11:18
@prout-bot
Copy link
Contributor

Seen on PROD (merged by @kelvin-chappell 5 hours, 10 minutes and 15 seconds ago) Please check your changes!

Sentry Release: prout

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.

3 participants