Skip to content

Comments

update: updated notation sign command based on spec#417

Merged
shizhMSFT merged 32 commits intonotaryproject:mainfrom
patrickzheng200:sign
Oct 31, 2022
Merged

update: updated notation sign command based on spec#417
shizhMSFT merged 32 commits intonotaryproject:mainfrom
patrickzheng200:sign

Conversation

@patrickzheng200
Copy link
Contributor

@patrickzheng200 patrickzheng200 commented Oct 26, 2022

This PR updates notation sign command based on the sign_spec.

Tested with notation certificate generate-test and notation sign on ACR.

Signed-off-by: Patrick Zheng patrickzheng@microsoft.com

@codecov-commenter
Copy link

codecov-commenter commented Oct 26, 2022

Codecov Report

❗ No coverage uploaded for pull request base (main@2992190). Click here to learn what that means.
The diff coverage is 50.00%.

@@           Coverage Diff           @@
##             main     #417   +/-   ##
=======================================
  Coverage        ?   34.11%           
=======================================
  Files           ?       21           
  Lines           ?     1061           
  Branches        ?        0           
=======================================
  Hits            ?      362           
  Misses          ?      690           
  Partials        ?        9           
Impacted Files Coverage Δ
cmd/notation/common.go 100.00% <ø> (ø)
cmd/notation/main.go 0.00% <0.00%> (ø)
cmd/notation/manifest.go 0.00% <0.00%> (ø)
cmd/notation/sign.go 37.23% <83.33%> (ø)
cmd/notation/verify.go 25.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@yizha1 yizha1 added this to the beta-1 milestone Oct 26, 2022
@yizha1 yizha1 linked an issue Oct 26, 2022 that may be closed by this pull request
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
@yizha1 yizha1 requested review from a team, priteshbandi, rgnote and shizhMSFT October 26, 2022 06:13
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Copy link
Contributor

@JeyJeyGao JeyJeyGao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yizha1 yizha1 requested a review from a team October 27, 2022 05:31
@yizha1 yizha1 removed this from the beta-1 milestone Oct 27, 2022
Patrick Zheng added 2 commits October 29, 2022 20:27
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Copy link
Contributor

@priteshbandi priteshbandi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Patrick Zheng added 2 commits October 31, 2022 14:17
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Patrick Zheng added 3 commits October 31, 2022 14:46
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Copy link
Contributor

@shizhMSFT shizhMSFT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@shizhMSFT shizhMSFT merged commit a08dc9e into notaryproject:main Oct 31, 2022
@patrickzheng200 patrickzheng200 deleted the sign branch October 31, 2022 08:05
7h3-3mp7y-m4n pushed a commit to 7h3-3mp7y-m4n/notation that referenced this pull request Mar 29, 2025
This PR updates notation sign command based on the
[sign_spec](https://github.com/notaryproject/notation/blob/main/specs/commandline/sign.md).

Tested with `notation certificate generate-test` and `notation sign` on
ACR.

Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
FeynmanZhou pushed a commit to FeynmanZhou/notation that referenced this pull request May 15, 2025
This PR updates notation sign command based on the
[sign_spec](https://github.com/notaryproject/notation/blob/main/specs/commandline/sign.md).

Tested with `notation certificate generate-test` and `notation sign` on
ACR.

Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Update notation CLI spec to support COSE as signing envelope (repo: notation) Implementing notation sign command

6 participants