-
Notifications
You must be signed in to change notification settings - Fork 558
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
Resolves #3088: version
sub-command expected behaviour documentation and testing
#3447
Resolves #3088: version
sub-command expected behaviour documentation and testing
#3447
Conversation
partially addresses sigstore#3088 Signed-off-by: Omri Bornstein <[email protected]>
Signed-off-by: Omri Bornstein <[email protected]>
Signed-off-by: Omri Bornstein <[email protected]>
version
sub-command expected behaviour documentation and testingversion
sub-command expected behaviour documentation and testing
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.
Thanks so much! Just a few small comments
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3447 +/- ##
==========================================
+ Coverage 30.38% 40.09% +9.70%
==========================================
Files 155 155
Lines 9982 9982
==========================================
+ Hits 3033 4002 +969
+ Misses 6497 5494 -1003
- Partials 452 486 +34 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Omri Bornstein <[email protected]>
Signed-off-by: Omri Bornstein <[email protected]>
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.
Thanks again!
Signed-off-by: Omri Bornstein <[email protected]>
Head branch was pushed to by a user without write access
Signed-off-by: Omri Bornstein <[email protected]>
Summary
cosign version
output #3088version
sub-commandversion
sub-command outputRelease Note
NONE
Documentation
The documentation of the
version
sub-command was updated such that the expected behaviour of printing to STDOUT is described.