A modern programming language that makes developers happier.

kotlinlang

:rocket: <https://github.com/krzema12/github-actions-kotlin-dsl/releases/tag/v0.25.0|v0.25.0 released!>
The highlight of this release is moving away from handcrafting YAML, and delegating it to <https://bitbucket.org/snakeyaml/snakeyaml/|snakeyaml> instead - the same library that's used in <https://github.com/charleskorn/kaml|kaml>. This way several known issues with invalid YAML got fixed, and the library became simpler. It will result in slight formatting changes of your output YAMLs, and hopefully no regressions :slightly_smiling_face:

first regression :sweat: <https://github.com/krzema12/github-actions-kotlin-dsl/issues/416>

you may want to wait with upgrading to v0.25.0 until it's fixed

<@U0B8UKF8R> <@UBBFK40RZ> <@UB9K6R4JH> maybe you have a spare moment? :point_up: :pray:

i probably won't have to the time to decipher snakeyaml's source..

I have something: <https://bitbucket.org/snakeyaml/snakeyaml/pull-requests/17/541-allow-disabling-anchors-when-dumping>

problem with that approach is.. snakeyaml will crash with a stackoverflow exception when dealing with any kind of recursiveness

as long as that is properly communicated / documented though..

Yeah, already pointed out by the author in the PR, I'll add a test for it

Kaml doesn't output anchors like that, so I'd guess it is possible with pure Snakeyaml too.

I started to factor out my DAOs into a standalone lib, to eventually make it usable in your lib, just need a bunch more of tests for it and have limited time, but I just created a
```on:
  "pull_request": {}
  "push": {}```
in a quick test without problem or special handling

To be frank, I'm still not convinced that kaml is the best choice here. I'd appreciate a PoC that would show how your DAOs would be used in the DSL lib

I find snakeyaml pretty convenient, not counting this anchors issue that's likely to be fixed soon-ish

Don't worry, I'm not just doing it for you :slightly_smiling_face:

<@UBBFK40RZ> got a moment to check out <https://bitbucket.org/snakeyaml/snakeyaml/pull-requests/17/541-allow-disabling-anchors-when-dumping#comment-324904093|https://bitbucket.org/snakeyaml/snakeyaml/pull-requests/17/541-allow-disabling-anchors-when-dumping#comment-324904093>?

got no account on bitbucket.. and when i tried to import the mavne project into idea there were a few warnings about inability to resolve plugins..
what can i do to help ?

that means.. catching serialization in snakeyaml when it runs into recusion .. and throwing ?

not sure i have the mental capacity for that today

Probably something along these lines. It's fine, no forcing from my side :) it would be cool to tackle it early next week so that the feature gets into their next release (also next week). I'll also try to find time for it, but no guarantees

What actually worries me most is snakeyaml's infrequent release schedule, like every half a year from what I've seen

Plan B is to temporarily copy snakeyaml's code into the DSL lib, with the feature added just to address the anchors issue. Then, promptly merging the PR in snakeyaml becomes less critical

fix is merged, I'll release it together with next usual release, that is probably on Friday