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.

Sign up for GitHub

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

Jump to bottom

Config precendence seems to be very broken at the moment #103

Open
spkane opened this issue Jan 21, 2022 · 1 comment
Open

Config precendence seems to be very broken at the moment #103

spkane opened this issue Jan 21, 2022 · 1 comment
Labels
bug Something isn't working next Higher priority issues

Comments

@spkane
Copy link
Contributor

spkane commented Jan 21, 2022

At the moment, it appears like at least some configuration (maybe all), is being completely overridden (or maybe only set) via the config file. This was noticed with the --interactive command line arg for cludo.

In theory the precedence should (and used to) work like this:

  • read the config file
  • overwrite with environment variables
  • command line arguments win
@tammersaleh
Copy link
Member

The expected behavior you described is correct to me. Sounds like we have a bug.

@spkane spkane added bug Something isn't working next Higher priority issues labels Feb 9, 2022
@goonzoid goonzoid mentioned this issue Feb 10, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working next Higher priority issues
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tammersaleh @spkane