Skip to content

Conversation

@czpilar
Copy link
Contributor

@czpilar czpilar commented Jan 24, 2026

Resolves #1284

@czpilar czpilar force-pushed the patches/1284-mixup-of-option-short-and-long-name branch from 0a99b60 to 02a853f Compare January 24, 2026 18:31
Signed-off-by: czpilar <david@czpilar.net>
@czpilar czpilar force-pushed the patches/1284-mixup-of-option-short-and-long-name branch from 02a853f to 5dee898 Compare January 24, 2026 21:32
@fmbenhassine
Copy link
Contributor

Thank you for the PR! LGTM 👍 I would just rename CommandContext#getOptionByFilter to getOptionByPredicate since we are providing a predicate as a parameter. I will amend that on merge.

@fmbenhassine
Copy link
Contributor

Rebased and merged as 7dd7801. Thank you for your contribution!

PS: please use your real name in your git configuration

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.

Mixup of Option short and long name

2 participants