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
It is common to "override" configured values from other sources such as the system properties or environment variables. Now that #55 is implemented, ConfigurationNodes can be dynamic and even represent multiple types ("10" can be both the string "10" and the number 10).
This used to exist until it was reverted in 285a2a5 due to a bad implementation that didn't combine different sources.
Proposal
Allow ConfigurationBuilder.source(ConfigurationSource) calls to be chained to add additional sources. Deserializers will then receive a merged configuration that combines all sources.
// the first source is prioritized - this is what the original implementation did, but was it the right choice?MyConfigurationconfiguration = Warp.builder(MyConfiguration.class)
.source(props)
.source(yaml)
.build();
assertconfiguration.property().equals("props");
Implementation
Implement a composite ConfigurationNode that combines several nodes then use that when calling the configuration deserializer.
The text was updated successfully, but these errors were encountered:
Motivation
It is common to "override" configured values from other sources such as the system properties or environment variables. Now that #55 is implemented,
ConfigurationNode
s can be dynamic and even represent multiple types ("10" can be both the string"10"
and the number10
).This used to exist until it was reverted in 285a2a5 due to a bad implementation that didn't combine different sources.
Proposal
Allow
ConfigurationBuilder.source(ConfigurationSource)
calls to be chained to add additional sources. Deserializers will then receive a merged configuration that combines all sources.property = props
Implementation
Implement a composite
ConfigurationNode
that combines several nodes then use that when calling the configuration deserializer.The text was updated successfully, but these errors were encountered: