Repository navigation
Support adding the commit-queue label before the wait time is over #45405
Description
Activity
Having the commit-queue
PRs queued for automated landing through the Commit Queue. label added before a PR is actually ready to land forces the scheduled GHA to do a lot more of work (it now has to fetch the full commit history, install ncu, etc.) which can delay the launch of regular CI runs on other PRs if we've reached the max number of active runners. For example, it's already possible to add the label while the CI is still running, but I've recommended against doing that because of that (and also we never know if the CI will be successful or not).If there was a way of doing that without the need of fetching the entire commit history, that could be interesting. Otherwise, there's always the author ready
PRs with CI started, the required approvals, and no outstanding review comments. label.why do we need the full commit history?
node/.github/workflows/commit-queue.yml
Lines 53 to 55 in 7ed44db
Reacted by Moshe Atlow- added a commit that references this issue
on Nov 12, 2022 - added a commit that references this issue
on Nov 21, 2022
Maybe this needs to be implemented mostly in
node-core-utils, but I'd like to open the discussion here first for visibility.I think we should be able to add the
commit-queuelabel to a PR when the only missing requirement for it to land is the wait time. It is always annoying to have to calculate the remaining time and then remember to come back to a pull request just to add this label.Is it desirable and implementable?
@nodejs/actions