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
We currently allow the default viper extensions - {"json", "toml", "yaml", "yml", "properties", "props", "prop", "hcl", "tfvars", "dotenv", "env", "ini"}. Many of these are inconvenient to use in the context of Zarf and supporting all of them adds complexity to a user by introducing unnecessary choices.
Additionally, Zarf may add custom parsing logic to config files to enable case sensitive map keys, which Viper does not support by default, see - zarf-dev/proposals#16.
The argument could be made to eliminate all but one choice. However, toml is useful for removing the layers of tabs that exist in yaml. Especially since configuration options tend to have three+ layers such as package.deploy.set. Yaml is useful for specifying maps and keeping parity with Helm charts. Many times the same a user will want to specify a value from a Helm values file, and keeping the option to copy and paste is convenient.
The text was updated successfully, but these errors were encountered:
We currently allow the default viper extensions -
{"json", "toml", "yaml", "yml", "properties", "props", "prop", "hcl", "tfvars", "dotenv", "env", "ini"}
. Many of these are inconvenient to use in the context of Zarf and supporting all of them adds complexity to a user by introducing unnecessary choices.Additionally, Zarf may add custom parsing logic to config files to enable case sensitive map keys, which Viper does not support by default, see - zarf-dev/proposals#16.
The argument could be made to eliminate all but one choice. However, toml is useful for removing the layers of tabs that exist in yaml. Especially since configuration options tend to have three+ layers such as
package.deploy.set
. Yaml is useful for specifying maps and keeping parity with Helm charts. Many times the same a user will want to specify a value from a Helm values file, and keeping the option to copy and paste is convenient.The text was updated successfully, but these errors were encountered: