notaryproject / notation-go

A collection of libraries for supporting sign and verify OCI artifacts. Based on Notary Project specifications.
Apache License 2.0
39 stars 42 forks source link

fix: updated error message of errExceededMaxVerificationLimit #314

Closed Two-Hearts closed 1 year ago

Two-Hearts commented 1 year ago

This PR updates error message of errExceededMaxVerificationLimit to be more user friendly.

FeynmanZhou commented 1 year ago

The error message looks good to me.

codecov-commenter commented 1 year ago

Codecov Report

Merging #314 (e0314a0) into main (39c8ed0) will increase coverage by 0.36%. The diff coverage is 100.00%.

:exclamation: 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     #314      +/-   ##
==========================================
+ Coverage   74.41%   74.78%   +0.36%     
==========================================
  Files          23       23              
  Lines        2177     2177              
==========================================
+ Hits         1620     1628       +8     
+ Misses        438      433       -5     
+ Partials      119      116       -3     
Impacted Files Coverage Δ
notation.go 64.48% <100.00%> (+3.73%) :arrow_up:

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