search

arcticicestudio / snowsaw

A lightweight, plugin-driven and dynamic dotfiles bootstrapper.
MIT License
98 stars 8 forks source link

Correct configuration state priority merging #68

Closed arcticicestudio closed 5 years ago

arcticicestudio commented 5 years ago

Epic: #33

To ensure loaded data from configuration files override default application values and CLI flags always take precedence over all other loaded states the overall loading logic must be refactored. It can be achieved by implementing a command initialization function (called before each command) that merges the configuration states in the correct order. Also the loading logic will be moved from pkg/config/constants.go into the pkg/config/config.go file to better separate the purpose of both files.

arcticicestudio commented 5 years ago

Resolved in 503055c63f3f0339e9554605849d1423639ea5ad