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.

Sign up for GitHub

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

Jump to bottom

Post helm chart linting output to the PR #580

Open

jforest opened this issue Oct 10, 2024 · 0 comments · Fixed by #590

Assignees

Contributor

jforest commented Oct 10, 2024 •

edited

Loading

Only post the changed charts, no need to post all.

Re-opening because I had to roll back the previous verison of this. #597 removed the posting to the PR.
#590 was the PR to add posting to the PR.

We ended up having failures due to what I believe is our lack of sanitization of the linting output before it is posted to the PR.

I think we need to find a good markdown sanitizer action or something to get us a safe output to use in posting to the PR.

jforest self-assigned this

jforest mentioned this issue

Post the lint results for changed charts to the PR #590

Merged

jforest closed this as completed in #590

jforest reopened this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment