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
{{ message }}
This repository has been archived by the owner on Dec 26, 2023. It is now read-only.
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.json
like this: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.This validation can be performed in this method:
https://github.com/ergomake/layerform/blob/a4f6211eb76c602ca5ad8f6429d0afa531499d48/internal/layerfile/layerfile.go#L37
The text was updated successfully, but these errors were encountered: