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

sdkman support #1299

Merged
merged 11 commits into from
Aug 7, 2024
Merged

sdkman support #1299

merged 11 commits into from
Aug 7, 2024

Conversation

prabhu
Copy link
Contributor

@prabhu prabhu commented Aug 7, 2024

New java version types powered by sdkman.

Examples:

-t java8
-t java11

cc: @heubeck @setchy @almaz045

Signed-off-by: Prabhu Subramanian <[email protected]>
@setchy
Copy link
Member

setchy commented Aug 7, 2024

So this would allow specifying the version of Java to use, or, only BOM if the repo/project is a certain version of Java?

As an aside, I feel like the project type flag is getting quite overloaded. Not too sure on a more elegant solution ATM 🤔

Signed-off-by: Prabhu Subramanian <[email protected]>
@prabhu
Copy link
Contributor Author

prabhu commented Aug 7, 2024

Version of java to install and use. Good point about project types. Maybe one day people will express the build environments using CycloneDX formulation, which we could reuse.

Signed-off-by: Prabhu Subramanian <[email protected]>
Signed-off-by: Prabhu Subramanian <[email protected]>
Signed-off-by: Prabhu Subramanian <[email protected]>
Signed-off-by: Prabhu Subramanian <[email protected]>
Signed-off-by: Prabhu Subramanian <[email protected]>
Signed-off-by: Prabhu Subramanian <[email protected]>
Signed-off-by: Prabhu Subramanian <[email protected]>
@prabhu prabhu merged commit ea0665d into master Aug 7, 2024
25 checks passed
@prabhu prabhu deleted the feature/sdkman-support branch August 7, 2024 19:31
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.

2 participants