-
Notifications
You must be signed in to change notification settings - Fork 365
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 helm lint action #1589
Add helm lint action #1589
Conversation
Seeing as it does not work. I posted a comment to an issue someone else has reported too. helm/chart-testing-action#108 |
@pglombardo is this something that you would accept now that it works? |
Absolutely and much appreciated! Give me a bit to get some time and go through the PR (and learn the Helm basics). |
da8f5da
to
679e6de
Compare
Looks good to me @MindTooth. Should we (or is there any value) in publishing these Helm charts to ArtifactHub? |
Maybe host ourself using https://github.com/helm/chart-releaser? Can then forgo any need for auth against ArtifactHub. 🤷🏻♂️ |
We should also update it with all parameters as of late. Can do that when this is merged. Start to treat helm as a valid option. |
a956ccc
to
a0efbba
Compare
a0efbba
to
567a261
Compare
This is slick now that I've had the time to look at it 😄 Thanks @MindTooth |
Description
Second attempt with newer version. Supersedes #1569.
Related Issue
Type of Change
Checklist
rake test
)