bugfix: ignore extra fields in yaml

canonical / chisel

GNU Affero General Public License v3.0

248 stars 39 forks source link

bugfix: ignore extra fields in yaml #107

Closed letFunny closed 7 months ago

letFunny commented 7 months ago

[x] Have you signed the CLA?

Ignoring extra fields makes the code forward compatible with changes in the yaml format, without a version bump being necessary.

letFunny commented 7 months ago

@cjdcordeiro You are absolutely right, I have added another field in the test to reflect that.

cjdcordeiro commented 7 months ago

@cjdcordeiro You are absolutely right, I have added another field in the test to reflect that.

A bit too late, but also not too urgent: @letFunny if you add a new key under slices, and it is not a map, chisel will fail to unmarshal it into setup.yamlSlice.