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

feat(cli): print documentation if parameter is ignored/infered #3370

Closed

Conversation

CommanderStorm
Copy link
Contributor

Does your PR solve an issue?

sort of follow-up to #3352

Resolves #3348 (comment)
Thanks @thiagokoster did not notice that this is a bit confusing

@abonander
Copy link
Collaborator

An explicit -r flag should always override the inferred behavior.

@CommanderStorm
Copy link
Contributor Author

CommanderStorm commented Jul 30, 2024

Are you sure?
That would mean that this is bug:
image

@abonander
Copy link
Collaborator

That's not at all what I was talking about there, did you link the right thing?

@CommanderStorm
Copy link
Contributor Author

Ah sorry, only skimmed over said PR

@abonander
Copy link
Collaborator

But yes, the current behavior is a bug. The point of the inference was to be a sane default, not a hard rule.

@abonander
Copy link
Collaborator

@CommanderStorm closing because this isn't the right fix, but I appreciate the initiative you've shown. Let's go back to #3348 to regroup.

@abonander abonander closed this Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Combining command options does not work
2 participants