-
Notifications
You must be signed in to change notification settings - Fork 858
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
Add short name for crds #1434
Add short name for crds #1434
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1434 +/- ##
==========================================
+ Coverage 51.50% 51.51% +0.01%
==========================================
Files 81 81
Lines 8013 8017 +4
==========================================
+ Hits 4127 4130 +3
- Misses 3386 3387 +1
Partials 500 500
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
Thanks for your contributions
f84b349
to
46b32a5
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.
After some research, that short name conflicts with other CRD can cause confusion and bugs, so we should only add necessary ones.
46b32a5
to
a0a19a5
Compare
a0a19a5
to
8c87e90
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.
Cool!
8c87e90
to
7d31d84
Compare
Add short name for crds as follows