-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
test(manager/helmv3): migrate from fs.readFile to fs.readLocalFile #13832
test(manager/helmv3): migrate from fs.readFile to fs.readLocalFile #13832
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs lint fixes too
Hi @viceice the build has failed on |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it was a temporary issue. This needs deconflicting now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs snapshot updates
Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
Hi @viceice I see some strange CodeQL build error, but I have no idea what could create it... |
🎉 This PR is included in version 31.64.1 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Changes:
migrate from fs.readFile to fs.readLocalFile (helmv3) #12987
Context:
Part of #7049, related to PR #12987
Documentation (please check one with an [x])
How I've tested my work (please tick one)
I have verified these changes via: