Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Jul 31, 2024

This PR makes refactoring of revocation with the updated notation-core-go.

Patrick Zheng added 3 commits July 30, 2024 12:38
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>
@codecov
Copy link

codecov bot commented Jul 31, 2024

Codecov Report

Attention: Patch coverage is 92.40506% with 6 lines in your changes missing coverage. Please review.

Project coverage is 80.86%. Comparing base (c3b2f51) to head (9340a5f).

Files Patch % Lines
verifier/verifier.go 92.40% 3 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #429      +/-   ##
==========================================
+ Coverage   80.74%   80.86%   +0.12%     
==========================================
  Files          33       33              
  Lines        2457     2483      +26     
==========================================
+ Hits         1984     2008      +24     
- Misses        331      332       +1     
- Partials      142      143       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ghost ghost added this to the 1.2.0 milestone Jul 31, 2024
@ghost ghost added the enhancement New feature or request label Jul 31, 2024
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
@ghost ghost removed the enhancement New feature or request label Aug 1, 2024
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Patrick Zheng added 2 commits August 6, 2024 16:50
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Patrick Zheng added 4 commits August 7, 2024 08:19
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
@ghost ghost requested a review from shizhMSFT August 8, 2024 02:37
@ghost ghost requested a review from JeyJeyGao August 8, 2024 02:37
JeyJeyGao
JeyJeyGao previously approved these changes Aug 8, 2024
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

shizhMSFT
shizhMSFT previously approved these changes Aug 8, 2024
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

Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
@ghost ghost dismissed stale reviews from shizhMSFT and JeyJeyGao via 9340a5f August 12, 2024 01:28
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

Copy link
Contributor

@rgnote rgnote left a comment

Choose a reason for hiding this comment

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

LGTM

@ghost ghost merged commit 3c5a659 into notaryproject:main Aug 14, 2024
@ghost ghost deleted the revocation branch August 14, 2024 00:47
@ghost ghost mentioned this pull request Aug 16, 2024
6 tasks
This pull request was closed.
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.

4 participants