Add a method to trigger reading of all config files #862
Conversation
…so that the last read config file takes precedence. This is at least interesting when subcommand aliases are used and the same option names exist in different configuration files for these different subcommand names. This allows a combined option set to be constructed from different subcommand names.
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #862 +/- ##
==========================================
- Coverage 99.45% 99.41% -0.05%
==========================================
Files 18 18
Lines 4069 4088 +19
==========================================
+ Hits 4047 4064 +17
- Misses 22 24 +2
... and 2 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
|
I made three pull requests today. Perhaps they will be accepted to be integrated into the main branch of CLI11. I use a CLI11 patched like this heavily in SNode.C to manage the whole configuration system. The changes have proven to be stable and functional. Please contact me in case you have questions about that pull requests. Kind regards |
|
Is the main purpose of this to allow reading of the config files in reverse order? If that is the case there probably is an argument to be made that that should be the proper way of doing it. In which case would the new method be necessary? |
use it for the default in `set_config` and simplify and add more flexibility to the the config processing, and potentially in other options as well. The reverse policy returns a vector but in reversed order from normal. This is what we want in the config processing Inspired by #862, and updated with recent code changes. --------- Co-authored-by: Volker Christian <me@vchrist.at> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
|
Going to close this unless @VolkerChristian has some other comment |
This pull request did not change the behavior of application code if the newly introduced method
App *all_config_files(bool all = true)is not used.App *all_config_files(bool all = true)to trigger reading of all config files in reverse order so that the last read config file takes precedence.