-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
【optimization】Use URL.Redacted method repleace redacted #18716
【optimization】Use URL.Redacted method repleace redacted #18716
Conversation
Codecov Report
@@ Coverage Diff @@
## main #18716 +/- ##
==========================================
- Coverage 67.39% 67.38% -0.02%
==========================================
Files 980 980
Lines 106759 106730 -29
Branches 2665 2665
==========================================
- Hits 71955 71919 -36
- Misses 30941 30947 +6
- Partials 3863 3864 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
/label release-note/update |
0af0907
to
fc34f1c
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
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
7c06cfe
to
5339719
Compare
Excuse me, is there anything else I need to do for this PR? |
Signed-off-by: lengrongfu <[email protected]>
5339719
to
1a57815
Compare
@lengrongfu Thanks, no additional steps need to do, will merge it after CI passed. |
use URL.Redacted method replease redacted Signed-off-by: lengrongfu <[email protected]> Signed-off-by: Wilfred Almeida <[email protected]>
use URL.Redacted method replease redacted Signed-off-by: lengrongfu <[email protected]>
Thank you for contributing to Harbor!
Comprehensive Summary of your change
Issue being fixed
Fixes #(issue)
release-note/update
Please indicate you've done the following: