GitHub's new "Merge Queue"

[vedantmgoyal9]

A new feature named "Merge Queue" has been released by GitHub. I would suggest Microsoft organization to atleast join the waiting list. It could be helpful for repositories like winget-pkgs.

More information here: https://github.blog/changelog/2021-10-27-pull-request-merge-queue-limited-beta/

[vedantmgoyal9]

cc @denelon

[vedantmgoyal9]

I think 🤔 then we will no longer have to depend on @msftbot for merging PRs, it will just have to add the PR to the queue and GitHub will take care of merging PRs.

[jedieaston]

depend on @msftbot for merging PRs

If there's API access for this feature (which may or may not exist in private beta, I don't know if it'll use the same merge endpoint or not), msftbot could be patched to be able to use it so the rest of the flow doesn't get broken. But that's not this team.

Tangentially related explanation: #13643 (comment)

[vedantmgoyal9]

Actually I was very eager to enroll myself but it needs me to be an admin of an organisation which I am not. Even if I create a new organisation, it needs the repo to have sufficient number of PRs which I cannot create in a short time. If @denelon could pass this issue internally, and get the organisation enrolled, we could test te feature and see if it can be useful here. However by only seeing the preview embedded in the post, I think that after when this is released, we don't have to "reapprove" PRs so that it can be merged.

[jedieaston]

we don't have to "reapprove" PRs so that it can be merged.

I've never had the problem described in the article, rather our problem (as I understand it) is that msftbot tries to internally calculate the merge before it "clicks the button". Most PRs stay green, but msftbot doesn't come to the same conclusion before the 1 minute timeout.

I don't know why msftbot has to separately calculate the merge (surely it can just ask GitHub's API "is this good to go?"), but if that's a hard requirement, then this won't solve our problem.

[vedantmgoyal9]

Most PRs stay green, but msftbot doesn't come to the same conclusion before the 1 minute timeout.

That's what I meant when I said "we don't have to "reapprove" PRs so that it can be merged."

[jedieaston]

But the reason we have to reapprove is to trigger msftbot to start its merging process, which is not the problem described in the article.

[vedantmgoyal9]

I can open a support ticket and ask for beta access (they've been very friendly in giving me acess for copilot and codespaces). I would verify it myself since I'm not clear with the information provided in the article.

[Trenly]

I agree with @jedieaston and don't think it resolves any issues with the current PR situation