Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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 added this to IGA Jul 13, 2023
@mhucka mhucka moved this to Todo in IGA Jul 13, 2023
@mhucka mhucka removed this from the 1.4.0 milestone May 15, 2024
@mhucka mhucka removed this from IGA May 15, 2024
@mhucka mhucka removed their assignment Sep 8, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Enhancement ✨ New feature or request Priority ★ Low priority
Projects
None yet
Development

No branches or pull requests

1 participant