Skip to content

feat: bump up notation-go to latest version#248

Merged
gokarnm merged 2 commits intonotaryproject:mainfrom
binbin-li:bump-up-notation-go
Jul 13, 2022
Merged

feat: bump up notation-go to latest version#248
gokarnm merged 2 commits intonotaryproject:mainfrom
binbin-li:bump-up-notation-go

Conversation

@binbin-li
Copy link
Copy Markdown
Contributor

What?

Fix the build failure due to the breaking change in notation-go as #247 mentioned.

Related changes in notation-go:
notaryproject/notation-go#77
notaryproject/notation-go#67
notaryproject/notation-go#64

Test

  1. Run go clean -modcache && go mod tidy
  2. Build cmd with no errors

Signed-off-by: Binbin Li libinbin@microsoft.com

Signed-off-by: Binbin Li <libinbin@microsoft.com>
Signed-off-by: Binbin Li <libinbin@microsoft.com>
@binbin-li binbin-li force-pushed the bump-up-notation-go branch from 450c6d0 to a90deff Compare July 13, 2022 04:59
Copy link
Copy Markdown
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

Once this PR is merged, we can close #234

Copy link
Copy Markdown
Contributor

@gokarnm gokarnm left a comment

Choose a reason for hiding this comment

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

LGTM!

@gokarnm gokarnm merged commit 5a323fa into notaryproject:main Jul 13, 2022
@binbin-li binbin-li mentioned this pull request Jul 14, 2022
7h3-3mp7y-m4n pushed a commit to 7h3-3mp7y-m4n/notation that referenced this pull request Mar 29, 2025
* feat: bump up notation-go to latest version

Signed-off-by: Binbin Li <libinbin@microsoft.com>
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.

3 participants