Skip to content
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

Change behavior of Config.Load method to fail if user-specified config file cannot be loaded #64

Open
atc0005 opened this issue Nov 25, 2020 · 0 comments
Assignees
Labels
config documentation Improvements or additions to documentation question Further information is requested
Milestone

Comments

@atc0005
Copy link
Owner

atc0005 commented Nov 25, 2020

Current behavior:

// Load attempts to first load the user-specified config file, then falls back
// to checking for a config file in the directory alongside the executable,
// then finally a config file from the user's configuration path. An error is
// returned if the configuration file cannot be loaded.

If the user specifies an exact path to a config file, presumably the intent is to override the use of any default locations? If that user-specified file cannot be loaded, shouldn't that be fatal?

@atc0005 atc0005 added documentation Improvements or additions to documentation question Further information is requested config labels Nov 25, 2020
@atc0005 atc0005 added this to the Future milestone Nov 25, 2020
@atc0005 atc0005 self-assigned this Nov 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
config documentation Improvements or additions to documentation question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant