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

ci: use Temurin distributions #476

Merged
merged 3 commits into from Mar 27, 2023
Merged

Conversation

yeikel
Copy link
Contributor

@yeikel yeikel commented Mar 22, 2023

Host runners include Temurin by default as part of the hosted tool cache

Using Temurin speeds up builds as there is no need to download and configure the Java SDK with every build.

Host runners include Temurin by default as part of the (hosted tool cache)(https://github.com/actions/setup-java/blob/main/docs/advanced-usage.md#hosted-tool-cache).Using Temurin speeds up builds as there is no need to download and configure the Java SDK with every build.

Co-authored-by: Moderne <team@moderne.io>
@brharrington
Copy link
Contributor

Looks like 15 is not available for Temurin, would likely need to bump that to 17 as it is the most recent LTS.

@yeikel
Copy link
Contributor Author

yeikel commented Mar 25, 2023

Looks like 15 is not available for Temurin, would likely need to bump that to 17 as it is the most recent LTS.

Does this project need to support Java 15?

@brharrington
Copy link
Contributor

15 was not an LTS releases and is no longer supported, so it can be bumped to 17.

@yeikel
Copy link
Contributor Author

yeikel commented Mar 27, 2023

15 was not an LTS releases and is no longer supported, so it can be bumped to 17.

I bumped it with d6340b2

@brharrington brharrington merged commit 7dbfafa into Netflix:master Mar 27, 2023
4 checks passed
@yeikel yeikel deleted the refactor/temurin branch March 27, 2023 18:37
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.

None yet

2 participants