SDK Man allows switching between SDKs, like JDK, with ease. It allows to pin the JDK to a particular version for a project/directory.
Adding a .sdkmanrc file to our projects would allow to automatically switch to the correct JDK when entering a project directory.
AC
| Title | Reference | Author | Source Branch | Dest Branch | |
|---|---|---|---|---|---|
| build: add .sdkmanrc to document the use of OpenJDK 8 | repos/maven/wmf-maven-tool-configs!9 | gehel | sdkman | main |
Will adopting SDKMan be a prescriptive change or just a default option? Would this change affect only a user's development environment, or also impact CI?
+1 for making SDKMan the default JDK manager, but it would be great if we could document an opt-out path.
Enforcing SDKMan could break existing workflows (e.g., Windows without WSL, or other version managers)
I think the proposal is to add .sdkmanrc files to all repo, but not to mandate that everyone uses them. Having an SDKMan config file doesn't force anyone to use it, in which case it just doesn't change anything for you.
Since there are no objection appart from the comment from @gmodena above, we can move forward with this, document and add config files to our repos.
gehel opened https://gitlab.wikimedia.org/repos/maven/wmf-maven-tool-configs/-/merge_requests/9
build: add .sdkmanrc to document the use of OpenJDK 8
amastilovic merged https://gitlab.wikimedia.org/repos/maven/wmf-maven-tool-configs/-/merge_requests/9
build: add .sdkmanrc to document the use of OpenJDK 8
Change #1055935 had a related patch set uploaded (by Gehel; author: Gehel):
[wikimedia-event-utilities@master] build: add SDKMan configuration
Change #1055935 merged by Ottomata:
[wikimedia-event-utilities@master] build: add SDKMan configuration
Change #1062712 had a related patch set uploaded (by Gehel; author: Gehel):
[analytics/gobblin-wmf@main] build: introduce .sdkmanrc to document JDK version to use for this project
Change #1098908 had a related patch set uploaded (by Gehel; author: Gehel):
[analytics/refinery/source@master] build: add sdkman configuration
Change #1062712 merged by Gehel:
[analytics/gobblin-wmf@main] build: introduce .sdkmanrc to document JDK version to use for this project
Change #1098908 merged by jenkins-bot:
[analytics/refinery/source@master] build: add sdkman configuration