Skip to content

Settings file discovery changed to prioritize user configuration

Marie Houillon requested to merge find_settings into master

What does this MR do?

The order of priority for discovering carputils settings.yaml was: CARPUTILS_SETTINGS > carputils root directory > working directory > $HOME/.config/carputils/

This Merge Request changes it to: CARPUTILS_SETTINGS > working directory > $HOME/.config/carputils/ > carputils root directory

As a consequence, a default settings file can be created in carputils root directory and then the user can overwrite it with one of the other options.

Author's checklist

Reviewer's checklist

All reviewers can help ensure accuracy, clarity, completeness, and adherence to the Contribution Guidelines.

Reviewer

  • Review by a code reviewer or other selected colleague to confirm accuracy, clarity, and completeness. This can be skipped for minor fixes without substantive content changes.

Maintainer

  • Review by assigned maintainer, who can always request/require the above reviews.
  • Ensure a release milestone is set.

Merge request reports

Loading