🐛 c.PlanRaw can be nil

[MadsBogeskov]

Hi Empowers!
I am getting a crash in Shuttle: https://github.com/lunarway/design-tokens/actions/runs/7009031830/job/19066620748?pr=17
The problem seems to be this piece of code in shuttleconfig.go:125 (Brilliantly found by reading the stacktrace)
The code crashing seems to be this:

switch c.PlanRaw {
    case false:
        // no plan
    default:
        c.Plan = c.PlanRaw.(string) // <-- this line to be specific

I assume this is a case that should be caught by the switch statement using what I assume is dark Go magic, instead of just doing a if c.PlanRaw == nil .

I am a bit unsure of what the right fix here is, besides just doing a nil check.

But this PR at least provides better logging in case this happens for someone else.

[mahlunar]

Thanks for the PR. Please fill in the description, so we know why the change was made when looking back at this in a few years.

[MadsBogeskov]

@mahlunar Hope this improves it.

[kjuulh]

Thanks for your contribution. This fix is enough for now. And should be fixed once we get yaml.v3 merged, which handles nulls a little differently =D