-
Notifications
You must be signed in to change notification settings - Fork 537
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
Fix up UI messages #2629
Fix up UI messages #2629
Conversation
Signed-off-by: Zachary Newman <zjn@chainguard.dev>
cmd/cosign/cli/sign/sign.go
Outdated
@@ -155,7 +152,7 @@ func SignCmd(ro *options.RootOptions, ko options.KeyOpts, signOpts options.SignO | |||
|
|||
var staticPayload []byte | |||
if signOpts.PayloadPath != "" { | |||
ui.Info(ctx, "Using payload from:", signOpts.PayloadPath) | |||
ui.Info(ctx, "Using payload from: %s", signOpts.PayloadPath) |
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.
Nit will be good to have this called InfoF to match similar pattern like PrintF SPrintF ...
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.
Done
Signed-off-by: Zachary Newman <zjn@chainguard.dev>
I wanted to add golanglint support via (It works when I run |
Codecov Report
@@ Coverage Diff @@
## main #2629 +/- ##
=======================================
Coverage 30.03% 30.04%
=======================================
Files 146 146
Lines 9283 9280 -3
=======================================
Hits 2788 2788
+ Misses 6065 6062 -3
Partials 430 430
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
* Fix up UI messages Signed-off-by: Zachary Newman <zjn@chainguard.dev> * Rename Warn/Info to Warnf/Infof Signed-off-by: Zachary Newman <zjn@chainguard.dev> Signed-off-by: Zachary Newman <zjn@chainguard.dev>
* Fix up UI messages Signed-off-by: Zachary Newman <zjn@chainguard.dev> * Rename Warn/Info to Warnf/Infof Signed-off-by: Zachary Newman <zjn@chainguard.dev> Signed-off-by: Zachary Newman <zjn@chainguard.dev>
Signed-off-by: Zachary Newman zjn@chainguard.dev
Summary
Release Note
Documentation