-
Notifications
You must be signed in to change notification settings - Fork 29.8k
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
tools: fix find-inactive actions #43377
tools: fix find-inactive actions #43377
Conversation
Review requested:
|
Fast-track has been requested by @LiviaMedeiros. Please 👍 to approve. |
We also probably want to revert using a hash instead of v1 as it looks like that doesn't work for some reason: https://github.com/nodejs/node/runs/6842110279?check_suite_focus=true |
Thanks for checking that! I think we should use gr2m/create-or-update-pull-request-action@6720400 instead. |
I just tried this PR over at https://github.com/nodejs/node-auto-test/runs/6842347772?check_suite_focus=true, and it opened nodejs/node-auto-test#48 just fine 👍 |
Landed in e227e52 |
PR-URL: #43377 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
PR-URL: #43377 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
PR-URL: #43377 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
PR-URL: #43377 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
PR-URL: #43377 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
PR-URL: nodejs/node#43377 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
No description provided.