Skip to content

Conversation

@shizhMSFT
Copy link
Contributor

@shizhMSFT shizhMSFT commented Jul 27, 2023

Signed-off-by: Shiwei Zhang <shizh@microsoft.com>
@shizhMSFT shizhMSFT marked this pull request as ready for review July 27, 2023 02:17
@shizhMSFT shizhMSFT requested a review from a user July 27, 2023 02:17
@shizhMSFT shizhMSFT added this to the 1.0.0 milestone Jul 27, 2023
@shizhMSFT shizhMSFT added dependencies Pull requests that update a dependency file go Pull requests that update Go code labels Jul 27, 2023
@shizhMSFT shizhMSFT mentioned this pull request Jul 27, 2023
15 tasks
Signed-off-by: Shiwei Zhang <shizh@microsoft.com>
Signed-off-by: Shiwei Zhang <shizh@microsoft.com>
@codecov-commenter
Copy link

Codecov Report

Merging #754 (820c916) into main (cd92b23) will not change coverage.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##             main     #754   +/-   ##
=======================================
  Coverage   63.98%   63.98%           
=======================================
  Files          40       40           
  Lines        2252     2252           
=======================================
  Hits         1441     1441           
  Misses        689      689           
  Partials      122      122           

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

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

@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

Copy link

@ghost ghost 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 merged commit f28cf32 into notaryproject:main Jul 28, 2023
@shizhMSFT shizhMSFT deleted the upgrade_dep branch July 28, 2023 01:59
@shizhMSFT shizhMSFT mentioned this pull request Jul 31, 2023
6 tasks
7h3-3mp7y-m4n pushed a commit to 7h3-3mp7y-m4n/notation that referenced this pull request Mar 29, 2025
- upgrade `notation-go` from `v1.0.0-rc.6` to `v1.0.0`
- upgrade dependencies for E2E tests
- update E2E test case to reflect
notaryproject/notation-go#340

---------

Signed-off-by: Shiwei Zhang <shizh@microsoft.com>
FeynmanZhou pushed a commit to FeynmanZhou/notation that referenced this pull request May 15, 2025
- upgrade `notation-go` from `v1.0.0-rc.6` to `v1.0.0`
- upgrade dependencies for E2E tests
- update E2E test case to reflect
notaryproject/notation-go#340

---------

Signed-off-by: Shiwei Zhang <shizh@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file go Pull requests that update Go code

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants