-
Notifications
You must be signed in to change notification settings - Fork 434
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 typo #1151
Fix typo #1151
Conversation
Hello, thank your for these changes. Please sign the CLA so we can proceed. |
I'm not sure if this kind of change would require bump up to the chart version? cc @TylerHelmuth |
@JaredTan95 any change inside a chart folder require a version change. |
@jdiegosierra pls following |
@jdiegosierra any update? |
@JaredTan95 Thank you for the info and sorry, I'll have a look today. |
@TylerHelmuth done :) |
@TylerHelmuth any idea of why those test are failing? Is happening the same in my other PR #1170 |
Likely a flaky test |
No description provided.