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.

Sign up for GitHub

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

Jump to bottom

Add useful info to AddPositionalValue panic. #77

Merged

integrii merged 1 commit into integrii:master from groutr:panic_info

Apr 23, 2022

Contributor

groutr commented Mar 30, 2022

Working on a somewhat complicated CLI and kept getting this vague panic about a positional value.

This PR inserts the names of the positional values into the panic message to assist the developer.


          Add useful info to AddPositionalValue panic.

integrii approved these changes

View reviewed changes

Owner

integrii left a comment

I like it! Thank you!

integrii merged commit 1465b3f into integrii:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment