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
fix: Fix the applicationset kind typo #12690
Conversation
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #12690 +/- ##
=======================================
Coverage 47.79% 47.79%
=======================================
Files 246 246
Lines 41968 41968
=======================================
Hits 20058 20058
Misses 19910 19910
Partials 2000 2000
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View 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.
Great catch!
@tossmilestone I'm a little hesitant to change that constant, because I'm not sure where/how it's used. Is the typo causing any particular issues? |
@crenshaw-dev I found this typo when I want to use the applicatinset kind in my code, and I have to define a constant instead to fix the error. I don't know if other people how to use the constant, but it's obviously an error if you use it as kind string. |
Signed-off-by: Shaw Ho <tossmilestone@gmail.com>
/cherry-pick release-2.4 |
/cherry-pick release-2.5 |
/cherry-pick release-2.6 |
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, @tossmilestone!
Cherry-pick failed with |
Signed-off-by: Shaw Ho <tossmilestone@gmail.com>
Signed-off-by: Shaw Ho <tossmilestone@gmail.com>
Signed-off-by: Shaw Ho <tossmilestone@gmail.com>
Manually cherry-picked onto release-2.4. |
Signed-off-by: Shaw Ho <tossmilestone@gmail.com>
Note on DCO:
If the DCO action in the integration test fails, one or more of your commits are not signed off. Please click on the Details link next to the DCO action for instructions on how to resolve this.
Checklist: