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

fix: fixed error messages of trust policy #326

Merged
merged 3 commits into from
Jun 26, 2023

Conversation

Two-Hearts
Copy link
Contributor

A quick PR to fix error messages of trust policy.

Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
@codecov-commenter
Copy link

codecov-commenter commented Jun 26, 2023

Codecov Report

Merging #326 (1986706) into main (d65bba3) will not change coverage.
The diff coverage is 100.00%.

❗ 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     #326   +/-   ##
=======================================
  Coverage   74.80%   74.80%           
=======================================
  Files          23       23           
  Lines        2203     2203           
=======================================
  Hits         1648     1648           
  Misses        437      437           
  Partials      118      118           
Impacted Files Coverage Δ
verifier/trustpolicy/trustpolicy.go 91.48% <100.00%> (ø)

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

@Two-Hearts Two-Hearts changed the title chore: fixed error messages of trust policy fix: fixed error messages of trust policy Jun 26, 2023
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

@priteshbandi priteshbandi merged commit 983e97d into notaryproject:main Jun 26, 2023
@Two-Hearts Two-Hearts deleted the chore branch June 27, 2023 00:09
@shizhMSFT shizhMSFT mentioned this pull request Jul 21, 2023
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.

5 participants