-
Notifications
You must be signed in to change notification settings - Fork 113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for cascading configurations #93
Comments
Currently we support the config source from:
As we have workspace supported, it makes sense to support
I think it should be relative to where it configured.
Good catch because items in replacement should be concat together instead of overwrited. I'm +1 for the first implementation proposal. It may also simplify the big section in |
I'm not seeing this deprecation warning. |
Now we'll load from all config sources, merging the result. Fixes crate-ci#93
The idea being that we merge the configs from the following locations
Use cases
Questions
Possible implementation
Option<_>
merge
functionunwrap_or
the field's defaultConfigSource
ofOption<&_>
ConfigFile
,ConfigArgs
etc toConfigSource
Config
struct that orders all of the potential sourcesConfig
struct with non-Option
membersConfig::from_sources(&[&dyn ConfigSource])
The text was updated successfully, but these errors were encountered: