Skip to content

git_graph: Add remote support - #55788

Merged
Anthony-Eid merged 2 commits into
mainfrom
git-graph-initial-data-remote-support
May 5, 2026
Merged

git_graph: Add remote support#55788
Anthony-Eid merged 2 commits into
mainfrom
git-graph-initial-data-remote-support

Conversation

@Anthony-Eid

@Anthony-Eid Anthony-Eid commented May 5, 2026

Copy link
Copy Markdown
Contributor

Follow up: #55167, #54468

This is the final PR for adding remote support on the git graph. It uses the client stream request support added in #55167 to add support for the initial graph data request. I also fixed a bug where GitGraph::FullyLoaded repository event was never emitted.

Self-Review Checklist:

  • I've reviewed my own diff for quality, security, and reliability
  • Unsafe blocks (if any) have justifying comments
  • The content is consistent with the UI/UX checklist
  • Tests cover the new/changed behavior
  • Performance impact has been considered and is acceptable

Closes ##53040

Co-authored-by: Remco Smits <djsmits12@gmail.com>

Release Notes:

  • git_graph: Add remote support

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label May 5, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label May 5, 2026
Co-authored-by: Ben Kunkle <ben@zed.dev>
@Anthony-Eid
Anthony-Eid added this pull request to the merge queue May 5, 2026
Merged via the queue into main with commit 4e08263 May 5, 2026
34 checks passed
@Anthony-Eid
Anthony-Eid deleted the git-graph-initial-data-remote-support branch May 5, 2026 21:47
ebaah46 pushed a commit to ebaah46/zed that referenced this pull request May 6, 2026
Follow up: zed-industries#55167,
zed-industries#54468

This is the final PR for adding remote support on the git graph. It uses
the client stream request support added in zed-industries#55167 to add support for the
initial graph data request. I also fixed a bug where
`GitGraph::FullyLoaded` repository event was never emitted.

Self-Review Checklist:

- [x] I've reviewed my own diff for quality, security, and reliability
- [x] Unsafe blocks (if any) have justifying comments
- [x] The content is consistent with the [UI/UX
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
- [ ] Tests cover the new/changed behavior
- [x] Performance impact has been considered and is acceptable

Closes #zed-industries#53040

Co-authored-by: Remco Smits \<djsmits12@gmail.com\>

Release Notes:

- git_graph: Add remote support

---------

Co-authored-by: Ben Kunkle <ben@zed.dev>
@ojpro

ojpro commented May 6, 2026

Copy link
Copy Markdown
Contributor

Thank you @Anthony-Eid

@kylekz kylekz mentioned this pull request May 8, 2026
3 tasks
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
Follow up: zed-industries#55167,
zed-industries#54468

This is the final PR for adding remote support on the git graph. It uses
the client stream request support added in zed-industries#55167 to add support for the
initial graph data request. I also fixed a bug where
`GitGraph::FullyLoaded` repository event was never emitted.

Self-Review Checklist:

- [x] I've reviewed my own diff for quality, security, and reliability
- [x] Unsafe blocks (if any) have justifying comments
- [x] The content is consistent with the [UI/UX
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
- [ ] Tests cover the new/changed behavior
- [x] Performance impact has been considered and is acceptable

Closes #zed-industries#53040

Co-authored-by: Remco Smits \<djsmits12@gmail.com\>

Release Notes:

- git_graph: Add remote support

---------

Co-authored-by: Ben Kunkle <ben@zed.dev>
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
Follow up: zed-industries#55167,
zed-industries#54468

This is the final PR for adding remote support on the git graph. It uses
the client stream request support added in zed-industries#55167 to add support for the
initial graph data request. I also fixed a bug where
`GitGraph::FullyLoaded` repository event was never emitted.

Self-Review Checklist:

- [x] I've reviewed my own diff for quality, security, and reliability
- [x] Unsafe blocks (if any) have justifying comments
- [x] The content is consistent with the [UI/UX
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
- [ ] Tests cover the new/changed behavior
- [x] Performance impact has been considered and is acceptable

Closes #zed-industries#53040

Co-authored-by: Remco Smits \<djsmits12@gmail.com\>

Release Notes:

- git_graph: Add remote support

---------

Co-authored-by: Ben Kunkle <ben@zed.dev>
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 staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants