UB
Universal Blue•2y ago
M2

For bluefin tags. Looks like we lost

For bluefin tags. Looks like we lost adding the tags to the bash array. https://github.com/ublue-os/bluefin/actions/runs/8605061498/job/23580493765#step:8:20
14 Replies
bsherman
bsherman•2y ago
oh?
M2
M2OP•2y ago
Yeah, we had a matrix variable previously it loos like that is not present anymore
bsherman
bsherman•2y ago
which one?
M2
M2OP•2y ago
matrix.is_latest_version matrix.is_stable_version
bsherman
bsherman•2y ago
that got moved
M2
M2OP•2y ago
Yes but it's referring to matrix.name instead of the variable name.
bsherman
bsherman•2y ago
instead of a matrix variable being used only to setup bash vars in the single "tags" step, its now a conditional in the "tags" step
M2
M2OP•2y ago
https://github.com/ublue-os/bluefin/actions/runs/8604817085/job/23579734249#step:8:37 As you can see it's being compared against an empty string. ${{ matrix.is_latest_version }} is empty
bsherman
bsherman•2y ago
ok, yeah, that's a bug 🙂 c'mon on thought you reviewed my PR 😉
M2
M2OP•2y ago
Yepp. I'm fixing right now
bsherman
bsherman•2y ago
thanks!
M2
M2OP•2y ago
Since we are in bash run right here. I can just use the bash variable name
bsherman
bsherman•2y ago
it should be clear, since the bash vars map to the old matrix vars thank you for catching this
M2
M2OP•2y ago

Did you find this page helpful?