Bug 239327 - Keep local commit history separated from PR-ed one
Summary: Keep local commit history separated from PR-ed one
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks: 239082
  Show dependency treegraph
 
Reported: 2022-04-14 02:57 PDT by Yusuke Suzuki
Modified: 2022-04-21 02:58 PDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Yusuke Suzuki 2022-04-14 02:57:13 PDT
My workflow with webkit-patch is

1. First, creating a topic branch.
2. Land random multiple commits frequently (it is pile of change xxx, update xxx.
3. Then, once all the diff becomes good (git diff main), run webkit-patch upload --update-changelogs to generate ChangeLogs and upload a patch to bugzilla.
4. I typically just commit ChangeLog change itself.
5. Periodically, I rebase all of the above commits based on main.
6. Once review is done, and some changes are requested, I change this branch and add further commits (like, fix xxx etc.)
7. Then, run webkit-patch upload --update-changelogs again to regenerate ChangeLog files and upload a patch.
8. When landing, use webkit-patch land -v.

However, when using `git webkit pr`, it uploads multiple commits as PR if I have multiple commits locally.
While I set history = when-user-owned option, it does not change.

Please create PR branch separately from the actual local working branch, and generate commit message automatically from ChangeLog (as we did in commit of webkit-patch).
Keeping local history is important since I would like to revert some part of attempt locally while modifying code.
Comment 1 Radar WebKit Bug Importer 2022-04-21 02:58:14 PDT
<rdar://problem/92084119>