-
Notifications
You must be signed in to change notification settings - Fork 57
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
make erorr more explicit #515
Conversation
pkg/cmd/push/cmd.go
Outdated
@@ -118,7 +118,7 @@ func runCommand(opts *pushOptions) func(*cobra.Command, []string) error { | |||
output.Infof("Pushing %s", opts.modelRef.String()) | |||
desc, err := PushModel(cmd.Context(), localRepo, remoteRepo, opts) | |||
if err != nil { | |||
return output.Fatalf("Failed to push: %s", err) | |||
return output.Fatalf("Failed to push: %s. Ensure that the organization and repository you are pushing to exist and that you have the necessary write permissions.", err) |
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.
Change
"Ensure that the organization and repository you are pushing to exist and that you have the necessary write permissions."
to
"Check you have write permissions to the organization and repository you are pushing to."
Please review. |
Message looks good now but I'd like @gorkem to review before we merge. Thanks @TusharMohapatra07! |
It is still showing that a change is requested. Please verify. |
Congratulations and thank you @TusharMohapatra07 for the PR! |
Thank you @bmicklea too for your patience and time |
Description
The error corresponding to
kit push
was unclear so this contribution is corresponding to making the error more explicit for end userLinked issues
#510