Skip to content

Fix coalescing of multi-line edits - #46220

Merged
maxbrunsfeld merged 1 commit into
mainfrom
ep-edit-coalescing
Jan 7, 2026
Merged

Fix coalescing of multi-line edits#46220
maxbrunsfeld merged 1 commit into
mainfrom
ep-edit-coalescing

Conversation

@maxbrunsfeld

Copy link
Copy Markdown
Collaborator

I was seeing some surprising un-coalesced edits in the edit history when looking at captured examples. I realized that we were only coalescing based on the ends of edit ranges, but I think we want to combine hunks where there are 8 or less unchanged lines in between changes. This PR implements that.

Release Notes:

  • N/A

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Jan 7, 2026
@maxbrunsfeld
maxbrunsfeld enabled auto-merge (squash) January 7, 2026 01:54
@maxbrunsfeld
maxbrunsfeld merged commit 67e2a23 into main Jan 7, 2026
24 checks passed
@maxbrunsfeld
maxbrunsfeld deleted the ep-edit-coalescing branch January 7, 2026 02:04
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
I was seeing some surprising un-coalesced edits in the edit history when
looking at captured examples. I realized that we were only coalescing
based on the *ends* of edit ranges, but I think we want to combine hunks
where there are 8 or less unchanged lines in between changes. This PR
implements that.

Release Notes:

- N/A
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
I was seeing some surprising un-coalesced edits in the edit history when
looking at captured examples. I realized that we were only coalescing
based on the *ends* of edit ranges, but I think we want to combine hunks
where there are 8 or less unchanged lines in between changes. This PR
implements that.

Release Notes:

- N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant