-
Notifications
You must be signed in to change notification settings - Fork 751
[stable-2.20] ci: fix issues identified by zizmor GHA linter #3331
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
[stable-2.20] ci: fix issues identified by zizmor GHA linter #3331
Conversation
Co-authored-by: Maxwell G <9920591+gotmax23@users.noreply.github.com>
Both is fine for me. |
I'll do it separately. I know it's an extra review but probably cleaner to separate the things. |
|
Thanks for the review and approval @felixfontein |
Backport to stable-2.17: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 8131b7a on top of patchback/backports/stable-2.17/8131b7ad32f9ec862473e7d8fd06358fc429815e/pr-3331 Backporting merged PR #3331 into stable-2.20
🤖 @patchback |
Backport to stable-2.18: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 8131b7a on top of patchback/backports/stable-2.18/8131b7ad32f9ec862473e7d8fd06358fc429815e/pr-3331 Backporting merged PR #3331 into stable-2.20
🤖 @patchback |
Backport to stable-2.19: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 8131b7a on top of patchback/backports/stable-2.19/8131b7ad32f9ec862473e7d8fd06358fc429815e/pr-3331 Backporting merged PR #3331 into stable-2.20
🤖 @patchback |
Co-authored-by: Maxwell G <9920591+gotmax23@users.noreply.github.com> (cherry picked from commit 8131b7a)
Co-authored-by: Maxwell G <9920591+gotmax23@users.noreply.github.com> (cherry picked from commit 8131b7a)
Co-authored-by: Maxwell G <9920591+gotmax23@users.noreply.github.com> (cherry picked from commit 8131b7a)
Manual backport of #3188
I would say it's sensible to backport these changes from the original PR. I don't think we need to integrate the zizmor session into the noxfile or add any of the comments related to zizmor. Totally open to suggestions there.
Also noticed
actions/checkout@v5on stable branches needs to be bumped tov6. Debating whether to do that in a separate PR or include it here.