Open jforest opened 1 month ago
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.
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.