-
Notifications
You must be signed in to change notification settings - Fork 505
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
Distribute ktlint via SDKMAN! #1363
Comments
If it can be fully automated, I would not have any objections against it. Please feel free to submit a PR. |
I'm currently a little bit sick but expect a PR somewhere around the end of the next week 😄 . |
@helpermethod We are ramping up to the 0.45.0 release in a couple of week max. So this would be a good time to submit your PR. |
@helpermethod We are ramping up to the 0.46.0 release in a couple of week max. So this would be a good time to submit your PR. |
ping @helpermethod |
Hi @jayvdb! I was occupied with some stuff at work and family, but I'll see if I can prepare the PR this evening :). The easiest way to use the Vendor API is via JReleaser, which automates most of the steps, and also enables other useful things like announcing new releases on Twitter, distribute releases via HomeBrew, Scoop, Chocolatey and many more. |
@helpermethod @jayvdb Just a heads-up, I am almost ready to request the build of the next release. So, if you want to get on board of this release train, you only have a couple of days left to submit and get a PR merged. |
Feel free to re-open once as PR is submitted. |
@helpermethod any plans to make this PR? |
@paul-dingemans I've started the process here sdkman/sdkman-db-migrations#639 Looks like you will need to request API credentials following this process sdkman/sdkman-db-migrations#639 I will follow up with a PR in this repo to add the sdkman-vendor-gradle-plugin to the release process. |
Hi @eygraber, if I remember correctly the problem with
|
That is being handled in the PR https://github.com/pinterest/ktlint/pull/1978/files#r1175844899 @paul-dingemans I think this will have to be done manually for 0.49.0; I can send you the zip on the Kotlin slack, or you can make it yourself (it's too large to attach here) and then manually add it to the 0.49.0 release. |
@shashachu Can you handle this? I don't think that I am priviliged to do so. |
@eygraber This just means a .zip artifact needs to be added to the 0.49.0 release? |
@shashachu yes, in a specific format. I can send it to you in slack (depending on what their size limits on file sharing are) or Google Drive |
@shashachu the PR on sdkman's side has been merged sdkman/sdkman-db-migrations#639 Did you receive credentials from them? |
I have checked with @shashachu yesterday. No credentials were yet received, so a new request has been sent. |
One of the sdkman maintainers mentioned that he sent it today. |
ktlint is not available through sdkman, even though the ktlint docs state it is. The sdkman docs do not list ktlint: https://sdkman.io/sdks Trying to install gives an error:
I am using the latest sdkman:
|
Sorry about that. Almost every release we had troubles with SDKMan. Eventually SDKMan has never been enabled after its last failure. I will remove it entire from release process and documentation. Apparently there are not a lot of users that depend on SDKMan for installing ktlint. Sorry, if you are one of those. |
Expected Behavior
It would be awesome if I could just use SDKMAN! to install the
ktlint
binary.Current Behavior
Currently
ktlint
can only be installed manually or via Homebrew.Additional information
To publish
ktlint
via SDKMAN!, the the binary would need to be packaged in a ZIP, with a top-level directory (the name does not actually matter) and abin
directory underneath, which would contain thektlint
binary, similar toNew versions could be published to SDKMAN! automatically via JReleaser
Wdyt? I also wouldn't mind helping out if you are interested!
The text was updated successfully, but these errors were encountered: