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

Switch to Eclipse Temurin and add SDKMan for local dev #1176

Merged
merged 1 commit into from
Jul 8, 2024

Conversation

artur-ciocanu
Copy link
Contributor

@artur-ciocanu artur-ciocanu commented Jul 8, 2024

Switch build from AdoptOpenJDK to Eclipse Temurin. Eclipse Temurin is the successor to AdoptOpenJDK and the team behind AdoptOpenJDK recommends switching to Eclipse Temurin. Here are some relevant links: https://blog.adoptopenjdk.net/2021/08/goodbye-adoptopenjdk-hello-adoptium/, https://projects.eclipse.org/projects/adoptium.

.sdkmanrc - this file is used by SDKMan to switch between different Java versions. This is especially useful when you are working on different repositories and each repository uses a different Java version. SDKMan will use .sdkmanrc to switch to the right Java version and set JAVA_HOME to the right path. Here is the official SDKMan docs: https://sdkman.io/jdks.

Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>
@artur-ciocanu
Copy link
Contributor Author

Hello @scottf could you please review this tiny PR.

Thank you 🙇‍♀️.

@scottf
Copy link
Contributor

scottf commented Jul 8, 2024

@artur-ciocanu Can you please provide some explanation for the reason to make this change. Please explain why Temurin over Adopt and then why add the ".sdkmanrc" file?

@artur-ciocanu
Copy link
Contributor Author

artur-ciocanu commented Jul 8, 2024

@artur-ciocanu Can you please provide some explanation for the reason to make this change. Please explain why Temurin over Adopt and then why add the ".sdkmanrc" file?

Response moved to main comment by @scottf

Copy link
Contributor

@scottf scottf left a comment

Choose a reason for hiding this comment

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

LGTM

@scottf scottf merged commit c2e2d18 into nats-io:main Jul 8, 2024
1 check passed
@artur-ciocanu artur-ciocanu deleted the use-temurin branch July 9, 2024 07:41
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.

2 participants