Skip to content

Conversation

@djeang
Copy link
Contributor

@djeang djeang commented Dec 19, 2024

No description provided.

Copy link
Member

@marc0der marc0der left a comment

Choose a reason for hiding this comment

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

The name and description are used on our website and when you run sdk list so we need a bit more verbosity and uniformity.

@djeang djeang requested a review from marc0der January 5, 2025 15:59
@djeang
Copy link
Contributor Author

djeang commented Jan 12, 2025

I’ve fixed the website URL as well. The description looks good to me. I think we can move forward.

Thank you!

@marc0der
Copy link
Member

Apologies, I totally missed your changes back in December and forgot about this PR. Would you like me to merge that now @djeang?

@djeang
Copy link
Contributor Author

djeang commented Jun 24, 2025

Hi Marc, sorry for this delay, too. I was focused on improving the product itself. I'm now ready to follow up the SdkMan onboarding. Can you merge the request please?

@marc0der marc0der merged commit a306728 into sdkman:master Jul 1, 2025
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