Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

revert: "feat: add support for signing blob (#379)" #411

Merged
merged 3 commits into from
May 31, 2024

Conversation

JeyJeyGao
Copy link
Contributor

@JeyJeyGao JeyJeyGao commented May 31, 2024

Revert:

Test:

  • added test for log package to pass 80% coverage target

This reverts commit ec42378.

Resolves part of #412
Signed-off-by: Junjie Gao junjiegao@microsoft.com

Copy link

codecov bot commented May 31, 2024

Codecov Report

Attention: Patch coverage is 90.62500% with 3 lines in your changes are missing coverage. Please review.

Please upload report for BASE (release-1.1@1a5b3e3). Learn more about missing BASE report.

Files Patch % Lines
notation.go 66.66% 1 Missing and 1 partial ⚠️
signer/plugin.go 95.23% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff               @@
##             release-1.1     #411   +/-   ##
==============================================
  Coverage               ?   80.19%           
==============================================
  Files                  ?       31           
  Lines                  ?     2101           
  Branches               ?        0           
==============================================
  Hits                   ?     1685           
  Misses                 ?      288           
  Partials               ?      128           

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

JeyJeyGao added 2 commits May 31, 2024 10:32
This reverts commit ec42378.

Signed-off-by: Junjie Gao junjiegao@microsoft.com
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
@JeyJeyGao JeyJeyGao marked this pull request as ready for review May 31, 2024 02:45
Copy link
Contributor

@Two-Hearts Two-Hearts 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

@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

@JeyJeyGao JeyJeyGao merged commit 94a0e13 into notaryproject:release-1.1 May 31, 2024
9 checks passed
@JeyJeyGao JeyJeyGao mentioned this pull request May 31, 2024
6 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants