Replies: 32 comments 35 replies
|
Same issue with rebase for me. |
|
Are signed commits required on your repos? I'm having the same issue. |
|
Running into the same issue in our repo as well |
|
My repo does not require signed commits (it's the single one in the org like this, I believe). The one outstanding thing I can think of is that the stack is based off a commit that is no longer the tip of |
|
I've had the same problem. My repo doesn't require signed commits. |
|
Same problem, we have stack with everything ready, but both squash or merge does not work. |
|
Luckily, for us it didn't get stuck when trying to merge the stack. However, it only merged the first (bottom) PR, then returned to "Ready" status for the other PRs. Since only one PR got merged, the "Rebase Stack" button became available again, but upon trying to rebase, it reported conflicts. Yet, there were no conflicts, as a local |
|
x-posting from #256 Hey all, thanks for the reports. We've identified a bug where approvals are not being appropriately evaluated when merging a stack of multiple PRs. We're actively investigating and will update when we've released a fix. In the meantime, you can get around this by merging one PR at a time. I know it's not ideal, but hopefully that will unblock your workflows in the short-term. Appreciate your patience! |
|
I’m seeing the same problem here :/ Since the public preview started, it hasn’t been working properly. |
|
Same issue here! Will you fix this in the short term? |
|
I have the same issue. |
|
Same here. And merging one by one doesn't work because it reruns the CI every time and drops PR approvals, so it's not a valid workaround. |
|
I have the same problem. It's a shame. |
|
I have this issue as well, which completely eliminates one of the main benefits of stacked PRs as they're unable to merge together atomically via a single merge group in the merge queue. My basic stacked PR setup
Observed behavior
Settings
Note that unlike others, my repository doesn't even require approvals or signed commits. |
|
I have found a life hack around this, create a new PR from based of the last PR of the stack pointing to the target branch of the first PR of the stack i.e. With merge commit turned on (not squash) it will allow you to merge all PRs in one go, with only needing the new PR to pass checks. I know we all joke about test in prod but GitHub made it a reality! |
|
The workaround for now seems to be merging each PR one by one. I've written https://github.com/tekumara/gh-stack-merge to automate this. It merges the stack one by one from the bottom up, waiting for checks to be ready on each PR before merging. |
|
I ran into this on my first stacked PR, and now I'm going back to not using them. This should have been a total showstopper that blocked a move to public preview. Could you please do another release announcement when this feature is usable? |
|
Same issue for me: first PR stack, 3 PRs. Stuck forever during squash+merge |
|
This is happening intermittently now. Sometimes it goes, sometimes it doesn't. |
|
Never works for me. |
|
same issue here: stuck on merging status when merging via ui button. When I run EDIT: it was a stack of 8 PRs... I merged them all by just merging the stack tip and closing the other 7 PRs. |
|
Same here 😞 |
|
In my case, this turned out to be a branch protection setting. Stack of 2 PRs, both approved by a write-access member, all checks green. The web UI sat on
Disabling My guess is that the automatic restack of the upper PR after the lower one lands rewrites its head SHA, which trips Two asks: (1) surface this error in the web UI instead of an endless spinner, and (2) consider not treating GitHub's own restack as a stale-making push. When the diff is unchanged and the author pushed nothing, dropping the approval defeats the point of merging a stack in one operation. |
|
Is there any update on this bug? |
|
@imanmahjoubi its been 2 weeks since your last post; could you please provide an update? even if the update is there is no update... feels like this has been forgotten about... |
|
Guys, please add some observability to this, it has been broken since day one for countless people on the most generic of things. Also it being unable to leave the merging state and failing to report that something is/was wrong is a really sad state of affairs. |
|
Any updates, please? My team is stopping using this feature (which seemed nice at first) because the last step is failing... |
|
I doubt they will update or fix this. The good developers all left github long ago. They are busy thinking of what new slop they can create rather than fixing what they couldn’t get working. |
|
yeah temporarily disabling "Dismiss stale pull request approvals when new commits are pushed" sorted this out for me, but this is literally the reason i embraced this feature (and bigged it up to my colleagues) - as it stands it's all a bit pointless 🤷 |



Uh oh!
There was an error while loading. Please reload this page.
Before:

After:

I've been stuck looking at these spinners for hours. Nothing is merging. I've cancelled and retried a few times; no luck.
All reactions