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 Tuesday, April 26, 2022 3:56:50 AM UTC
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
Radar WebKit Bug Importer
Comment 1 Tuesday, May 3, 2022 3:57:14 AM UTC
Elliott Williams
Comment 2 Tuesday, May 31, 2022 7:10:12 PM UTC
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.