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

Ordering of —log-dest and —mode matters, but shouldn't #9

Open
mhucka opened this issue May 22, 2023 · 0 comments
Open

Ordering of —log-dest and —mode matters, but shouldn't #9

mhucka opened this issue May 22, 2023 · 0 comments
Labels
Enhancement ✨ New feature or request Priority ★ Low priority

Comments

@mhucka
Copy link
Contributor

mhucka commented May 22, 2023

The way that Click parses arguments ends up causing the order of arguments to matter. For IGA, I ended up having to make --log-dest and --mode both "eager" arguments (in Click terminology); click parses eager arguments in the order they're given on the command line, which is a problem because changing the mode changes the details of how logging is done. If you pass --log-dest first, then --mode doesn't work as expected.

@mhucka mhucka added Bug 🐛 Something isn't working Priority ★★★ High priority labels May 22, 2023
@mhucka mhucka self-assigned this May 22, 2023
@mhucka mhucka added Priority ★ Low priority Enhancement ✨ New feature or request and removed Priority ★★★ High priority Bug 🐛 Something isn't working labels May 22, 2023
@mhucka mhucka changed the title Ordering of —log-dest and —mode matter, but shouldn't Ordering of —log-dest and —mode matters, but shouldn't Jun 3, 2023
@mhucka mhucka added this to the 1.4.0 milestone Jul 13, 2023
@mhucka mhucka removed this from the 1.4.0 milestone May 15, 2024
@mhucka mhucka removed their assignment Sep 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement ✨ New feature or request Priority ★ Low priority
Projects
None yet
Development

No branches or pull requests

1 participant