Post helm chart linting output to the PR
Open
@jforest is already working on this.
Since Oct 10, 2024.
- Dominant language
- Markdown
- Stars
- 46
- Forks
- 40
- Avg merge
- 4h 3m
- Merged PRs (30d)
- 6
Description
Only post the changed charts, no need to post all.
Re-opening because I had to roll back the previous verison of this. https://github.com/rstudio/helm/pull/597 removed the posting to the PR.
https://github.com/rstudio/helm/pull/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.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.