Skip to content

git: Don't call git2::Repository::find_remote for every blamed buffer#44107

Merged
cole-miller merged 2 commits intomainfrom
blame-less-git2
Dec 4, 2025
Merged

git: Don't call git2::Repository::find_remote for every blamed buffer#44107
cole-miller merged 2 commits intomainfrom
blame-less-git2

Conversation

@cole-miller
Copy link
Member

We already store the remote URLs for origin and upstream in the RepositorySnapshot, so just use that data. Follow-up to #44092.

Release Notes:

  • N/A

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Dec 3, 2025
@cole-miller cole-miller requested a review from Veykril December 4, 2025 01:10
@cole-miller cole-miller merged commit d5ed9d3 into main Dec 4, 2025
23 checks passed
@cole-miller cole-miller deleted the blame-less-git2 branch December 4, 2025 18:25
AlpSha pushed a commit to AlpSha/zed that referenced this pull request Dec 5, 2025
…er (zed-industries#44107)

We already store the remote URLs for `origin` and `upstream` in the
`RepositorySnapshot`, so just use that data. Follow-up to zed-industries#44092.

Release Notes:

- N/A
CherryWorm pushed a commit to CherryWorm/zed that referenced this pull request Dec 16, 2025
…er (zed-industries#44107)

We already store the remote URLs for `origin` and `upstream` in the
`RepositorySnapshot`, so just use that data. Follow-up to zed-industries#44092.

Release Notes:

- N/A
someone13574 pushed a commit to someone13574/zed that referenced this pull request Dec 16, 2025
…er (zed-industries#44107)

We already store the remote URLs for `origin` and `upstream` in the
`RepositorySnapshot`, so just use that data. Follow-up to zed-industries#44092.

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.

2 participants