NEW 239752
Support merging pull requests containing multiple commits 
https://bugs.webkit.org/show_bug.cgi?id=239752
Summary Support merging pull requests containing multiple commits
Elliott Williams
Reported 2022-04-25 19:56:50 PDT
Currently, it seems like we require PR branches to be squashed before merging. I think we should encourage contributors to write PRs with multiple commits by lifting this restriction and instead teaching Merge-Queue to do the squashing. I would expect it to use the PR description to write the squashed commit.
Attachments
Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2022-05-02 19:57:14 PDT
<rdar://problem/92648043>
Elliott Williams
Comment 2 2022-05-31 11:10:12 PDT
Real world example of where a workflow like this is needed: I'm relanding a change along with a fix. I think it's most logical, both for history and my reviewers, to land two commits: one is the automated `git revert` commit which undoes the revert, and the other applies fixes. Currently, I'm forced to squash them together. As a result, the merged commit doesn't make it clear what was changed.
Note You need to log in before you can comment on or make changes to this bug.