Joe Julian
Joe Julian
For an example of another PR taking this approach, see https://github.com/helm/helm/pull/9426#issuecomment-1407259188
Folks, if you've come down this far just to say "me too", please just add your :+1: to the initial description instead. Thanks!
Not according to GitHub
@pete-woods @yxxhero and I are just triage maintainers. You need a core maintainer to review.
Tested this and it works for me. I wish we could make it an error if it's past the unavailable deadline, but that would be a breaking change, I guess,...
> Since my commit should not have made major changes to cause the failed test run, I checked out the commit before it and ran the test suite there. I...
sorry for my force-pushes. I neglected to sign my commit and am trying to help get this across the finish line for this release.
Happy to have a contribution to address this. Probably should start with a [HIP](https://github.com/helm/community/blob/main/hips/hip-0001.md) also take a look at the [contributing](https://github.com/helm/helm/blob/main/CONTRIBUTING.md) doc.
@sabre1041 Here's a repro to use to validate this issue [foo.tar.gz](https://github.com/helm/helm/files/10841711/foo.tar.gz) Create the test file: ``` printf "test line 1\ntest line 2\n" > files/test.txt ``` This shows that the file...
Looking at the history of that function, it's been stated rather explicitly by the core maintainers that this is a result of a CNCF security team audit. > To be...