You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Then run layerform configure on this file, layerform will just panic. See video below:
configure-panic.mov
Solution
We should not assume that all layer listed in dependencies array always exists, we should check if that is the case and fail with a good error message to the user.
Bug
If I write a
layerform.jsonlike this:Then run
layerform configureon this file,layerformwill just panic. See video below:configure-panic.mov
Solution
We should not assume that all layer listed in
dependenciesarray always exists, we should check if that is the case and fail with a good error message to the user.This validation can be performed in this method:
https://github.com/ergomake/layerform/blob/a4f6211eb76c602ca5ad8f6429d0afa531499d48/internal/layerfile/layerfile.go#L37