-
Notifications
You must be signed in to change notification settings - Fork 42
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
Refactor to use notation-core-go's SignatureEnvelope #77
Conversation
@priteshbandi this PR is difficult to review because |
@qmuntal Updated PR to not move files from |
faf962b
to
3b5638e
Compare
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.
LGTM! Tracking open issues in #80 .
Signed-off-by: Pritesh Bandi <[email protected]>
Signed-off-by: Pritesh Bandi <[email protected]>
Signed-off-by: Pritesh Bandi <[email protected]>
Mistakenly closed while trying to resolving merge conflicts..Reopening |
* Refactor to use notation-core-go's SignatureEnvelope * move code from signature/jws to signature Signed-off-by: Pritesh Bandi <[email protected]> Signed-off-by: Junjie Gao <[email protected]>
Change Log
Dependency
Because of dependent changes build will fail. Once dependent PR is approved I will update this PR with updated go.sum.
Signed-off-by: Pritesh Bandi [email protected]