-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[Improvement] add resource full name check #15786
[Improvement] add resource full name check #15786
Conversation
1465bdf
to
a987c0b
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #15786 +/- ##
============================================
- Coverage 39.19% 39.19% -0.01%
Complexity 4876 4876
============================================
Files 1317 1317
Lines 45026 45037 +11
Branches 4806 4808 +2
============================================
+ Hits 17646 17650 +4
- Misses 25495 25501 +6
- Partials 1885 1886 +1 ☔ View full report in Codecov by Sentry. |
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.
|
Purpose of the pull request
add resource full name check to avoid
../
.Brief change log
ResourceServiceImpl.checkFullName
and UTVerify this pull request