search

sdkman / sdkman-action

SDKMAN! GitHub Action
22 stars 7 forks source link

Consider rewriting to TypeScript action #12

Open musketyr opened 2 years ago

musketyr commented 2 years ago

Hi, first, thanks again for including Pierrot in SDKMAN! I was considering creating my own action based on https://github.com/DeLaGuardo/setup-graalvm which also leverages caching but it might be easier if this action is rewritten instead. Is this something you would consider to make the action more efficient?

eddumelendez commented 1 year ago

Sorry for the long delay. This is a project which we contribute in our spare time. I have been thinking about how to fix it as-is (see https://github.com/sdkman/sdkman-action/pull/11) due to my lack of Typescript knowledge. If you are willing to do it, please go ahead, I will be so happy to accept the contribution. LMK if you need some help.

musketyr commented 1 year ago

Hi, no worries. I will try to take look soon. Let's keep this issue open.