-
Notifications
You must be signed in to change notification settings - Fork 542
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
nox: add actionlint to lint Github Actions workflows #1848
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.
brilliant work @gotmax23
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.
Thanks for adding this and for fixing those issues in the package docs build workflow @gotmax23
@gotmax23 I think this is ready to merge. Can you please add the relevant backport labels? I'm inclined to backport to stable-2.17 only but maybe we should keep this on devel until we cut stable-2.18. Thank you. |
I see no reason not to backport this to all stable branches and fix any issues that may come up there too. |
What do you think, @oraNod? Is there something in particular you're worried about? |
In any case, I'll merge this to devel now |
This PR integrates [actionlint](https://github.com/rhysd/actionlint) into our CI pipeline to check our Github Actions for best practices and common errors.
c0facbe
to
b5e957a
Compare
Rebasing on devel first |
@gotmax23 I'm not worried about anything necessarily but just wasn't sure. now that I think about it again I guess there's no reason not to backport. thanks for adding this and handling the merge. |
Sounds good, @oraNod. Thanks! |
Backport to stable-2.15: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 3b43d64 on top of patchback/backports/stable-2.15/3b43d6467dc97524a6e5184b069f4a590e9b4392/pr-1848 Backporting merged PR #1848 into devel
🤖 @patchback |
Backport to stable-2.17: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 3b43d64 on top of patchback/backports/stable-2.17/3b43d6467dc97524a6e5184b069f4a590e9b4392/pr-1848 Backporting merged PR #1848 into devel
🤖 @patchback |
Backport to stable-2.14: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 3b43d64 on top of patchback/backports/stable-2.14/3b43d6467dc97524a6e5184b069f4a590e9b4392/pr-1848 Backporting merged PR #1848 into devel
🤖 @patchback |
Backport to stable-2.16: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 3b43d64 on top of patchback/backports/stable-2.16/3b43d6467dc97524a6e5184b069f4a590e9b4392/pr-1848 Backporting merged PR #1848 into devel
🤖 @patchback |
Ah, this probably needs to be done manually because of the python-version conflicts... |
(cherry picked from commit 3b43d64)
(cherry picked from commit 3b43d64)
(cherry picked from commit 3b43d64)
(cherry picked from commit 3b43d64)
This PR integrates actionlint into our CI pipeline to check our Github Actions for best practices and common errors.