Piotr Krzemiński
01/20/2023, 9:37 AMVampire
01/20/2023, 9:50 AMPiotr Krzemiński
01/20/2023, 10:04 AMVampire
01/20/2023, 10:38 AM"null"
, which would still need special casing in your lib to differ between null
and "null"
.
I would have expected from snakeyaml
strategy:
matrix:
distribution:
- id: invalid
label: invalid
- id: ''
label: ''
- id: 'null'
label: 'null'
Actually false
and true
might also be affected.
At least the YAML schema validation is unhappy if there is false
or true
where a String
is expected.
Could be the same with other special values.Piotr Krzemiński
01/20/2023, 11:00 AMVampire
01/20/2023, 11:02 AMPiotr Krzemiński
01/20/2023, 11:02 AMVampire
01/20/2023, 11:20 AMexactly… 😅 anyway, I hope the project brings some value even in this stateDefinitely 🙂
role-to-assume: arn:aws:iam::12345678901234567890:role/github-actions-role/${{
github.token }}
which is identical to
role-to-assume: arn:aws:iam::12345678901234567890:role/github-actions-role/${{github.token }}
when it should have generated
role-to-assume: arn:aws:iam::12345678901234567890:role/github-actions-role/${{
\ github.token }}
to not loose the space.Piotr Krzemiński
01/20/2023, 11:54 AMVampire
01/22/2023, 4:58 PMdistribution = Distribution.Custom(expr("matrix.distribution.user-id")
So something like this would be marginally better / more type-safe:
distribution = Distribution.Custom(expr { matrix.distribution.userId })
But still quite some boilerplate that could maybe be made nicer by magic from your lib. 🙂Piotr Krzemiński
01/23/2023, 6:52 AMVampire
01/23/2023, 4:30 PMthanks @Vampire for fixing the YAML serialization bug! I'm curious if you'll be able to use DSL after this fix in Vampire/setup-wslIt's done. 🙂 https://github.com/Vampire/setup-wsl/commit/4eea7c92d6c56a39bfcebcdcb93be63388497f59
Piotr Krzemiński
01/23/2023, 5:24 PM