https://kotlinlang.org logo
#github-workflows-kt
Title
# github-workflows-kt
j

jmfayard

02/04/2022, 1:15 PM
p

Piotr Krzemiński

02/04/2022, 1:16 PM
if you tested the wrappers work fine with these versions, then it's fine 🙂 I think I asked about it somewhere in the comments
at some moment I asked you to switch from e.g.
v2.1.0
to
v2
but then I thought it may not actually work, my bad. I just ask to double check it in runtime
I'm working on generating the wrappers, then such problems will disappear (https://github.com/krzema12/github-actions-kotlin-dsl/issues/71)
j

jmfayard

02/04/2022, 1:19 PM
I will check that the wrappers work
👍 1
p

Piotr Krzemiński

02/04/2022, 4:08 PM
Do you have an example run of "RefreshVersions PR" workflow?
Yaml itself doesn't mean it will work :)
I see only "runOnGitHub" check
p

Piotr Krzemiński

02/04/2022, 4:55 PM
Ok, thanks! Approving
Thank you!
K 1
unfortunately it doesn't work: https://github.com/krzema12/test-repo/runs/5070645674?check_suite_focus=true, it says "Error: Unable to resolve action
peterjgrainger/action-create-branch@v2
, unable to find version `v2`"
I think that the new wrappers don't output exactly the same YAML as the versions are different. Other changes we see here are cosmetics
j

jmfayard

02/08/2022, 8:40 AM
wouldn't it make sense to add in the constructor of
Action
a call to:
curl -H "Accept: application/vnd.github.v3+json" <https://api.github.com/repos/peterjgrainger/action-create-branch/releases>
that would • throw an error if the tag doesn't exist • outputs a warning if a newer version of the Action is available • + adding an API to override the version, something like
Action.withVersion("2.1.6")
p

Piotr Krzemiński

02/08/2022, 8:43 AM
I'd like to handle version checking primarily in build time of the library, but what you say can be an alternative opt-in approach (especially with overriding the version, I thought about it). Could you create a feature request?
j

jmfayard

02/08/2022, 9:02 AM
Checking new versions is not so important I did a ticket about the rest
2 Views