Skip to content

Comments

update: Package signature refactoring#200

Merged
priteshbandi merged 31 commits intonotaryproject:mainfrom
patrickzheng200:sign
Nov 17, 2022
Merged

update: Package signature refactoring#200
priteshbandi merged 31 commits intonotaryproject:mainfrom
patrickzheng200:sign

Conversation

@patrickzheng200
Copy link
Contributor

@patrickzheng200 patrickzheng200 commented Nov 11, 2022

Files intended to be reviewed in this PR:

  1. All files inside the signer package

(Refactoring of signer package in this PR incorporates the updated plugin and plugin/proto packages.)

@codecov-commenter
Copy link

codecov-commenter commented Nov 12, 2022

Codecov Report

Merging #200 (1f11330) into main (ed31122) will decrease coverage by 8.25%.
The diff coverage is 25.29%.

@@            Coverage Diff             @@
##             main     #200      +/-   ##
==========================================
- Coverage   73.55%   65.29%   -8.26%     
==========================================
  Files          27       23       -4     
  Lines        1777     1576     -201     
==========================================
- Hits         1307     1029     -278     
- Misses        379      477      +98     
+ Partials       91       70      -21     
Impacted Files Coverage Δ
notation.go 32.58% <0.00%> (ø)
plugin/proto/sign.go 0.00% <ø> (ø)
verifier/verifier.go 73.12% <0.00%> (ø)
signer/plugin.go 22.67% <22.67%> (ø)
signer/signer.go 28.78% <28.78%> (ø)
internal/envelope/envelope.go 46.15% <46.15%> (ø)

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

Patrick Zheng and others added 14 commits November 12, 2022 15:16
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Temporary fix for
notaryproject/notation-core-go#88

Signed-off-by: rgnote <5878554+rgnote@users.noreply.github.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Patrick Zheng added 2 commits November 15, 2022 12:00
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Patrick Zheng added 3 commits November 16, 2022 09:35
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Patrick Zheng added 2 commits November 16, 2022 15:57
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Patrick Zheng added 3 commits November 17, 2022 09:59
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Patrick Zheng added 2 commits November 17, 2022 11:26
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

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

@priteshbandi priteshbandi merged commit 2573c88 into notaryproject:main Nov 17, 2022
@patrickzheng200 patrickzheng200 deleted the sign branch November 18, 2022 00:56
@ghost ghost mentioned this pull request Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants