Skip to content

Fix:(issue_2077) Make sure onUsageError is invoked for command when a… - #2081

Merged
dearchap merged 1 commit into
urfave:mainfrom
dearchap:issue_2077
Mar 31, 2025
Merged

dearchap merged 1 commit into
urfave:mainfrom
dearchap:issue_2077

Conversation

@dearchap

Copy link
Copy Markdown
Contributor

…rgs are in error

What type of PR is this?

(REQUIRED)

  • bug

What this PR does / why we need it:

(REQUIRED)

Which issue(s) this PR fixes:

(REQUIRED)

Fixes #2077

Special notes for your reviewer:

(fill-in or delete this section)

Testing

(fill-in or delete this section)

go test -run=TestArgumentsSubcommand

Release Notes

(REQUIRED)

OnUsageError() is invoked for errors in arg parsing as well. 

@dearchap
dearchap requested a review from a team as a code owner March 30, 2025 21:27
@dearchap
dearchap merged commit fe6ff48 into urfave:main Mar 31, 2025
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.

V3: Argument parsing errors do not invoke OnUsageError

2 participants