-
Notifications
You must be signed in to change notification settings - Fork 16
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
cli: api error handling #55
Conversation
d4c1b1b
to
9af3bb8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works well! It is really unfortunate that we have to rely on reflection to do this, but I don't think we can do otherwise.
Note that we should also use utils.HandleBasicApiError
in the logs
command.
75fa2f5
to
75b0491
Compare
75b0491
to
7a5630f
Compare
7a5630f
to
3faf7ca
Compare
Unfortunately, Cobra's Edit: See spf13/cobra#914 (comment) |
3faf7ca
to
87e208a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's very nice having all the error handling in a single place! Nice job 😄
closes #16