Skip to content

Conversation

@rgoldberg
Copy link
Contributor

Add ParsableArguments#_errorPrefix to allow a more-customized error prefix than ParsableArguments#_errorLabel.

_errorLabel wasn't tested or documented outside DocC, so I didn't do anything more for _errorPrefix.

Resolve #836.

Checklist

  • I've added at least one test that validates that my change is working, if appropriate
  • I've followed the code style of the rest of the project
  • I've read the Contribution Guidelines
  • I've updated the documentation if necessary

…refix than ParsableArguments#_errorLabel.

Signed-off-by: Ross Goldberg <[email protected]>
@rgoldberg
Copy link
Contributor Author

rgoldberg commented Dec 17, 2025

Rebased this on current main.

Thanks for merging #808. That will be very useful for my (and many other people's) completions.

@rauhul
Copy link
Contributor

rauhul commented Dec 17, 2025

Rebased this on current main.

Thanks for merging #808. That will be very useful for my (and many other people's) completions.

How does this help completions?

@rgoldberg
Copy link
Contributor Author

@rauhul This (#837) doesn't help completions. #808, which was just merged, helps completions.

I just wanted to thank you guys for merging #808.

I included the thanks in my prior comment on this PR (about having rebased this PR) since I was already writing a comment.

Sorry for any confusion.

Copy link
Member

@natecook1000 natecook1000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Someday we can look at how to customize errors/help more completely, but this is a fine change for now.

@natecook1000 natecook1000 enabled auto-merge (squash) December 17, 2025 17:07
@rgoldberg
Copy link
Contributor Author

@natecook1000 Thanks for the merges.

@natecook1000 natecook1000 merged commit a836797 into apple:main Dec 17, 2025
29 checks passed
@rgoldberg rgoldberg deleted the 836-error-prefix branch December 17, 2025 17:21
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.

Allow completely changing error output prefix

3 participants