Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

gitserver: Move CommitsUniqueToBranch to callsite#62514

Merged
eseliger merged 1 commit into
mainfrom
es/05-08-gitservermovecommitsuniquetobranchtocallsite
May 10, 2024
Merged

gitserver: Move CommitsUniqueToBranch to callsite#62514
eseliger merged 1 commit into
mainfrom
es/05-08-gitservermovecommitsuniquetobranchtocallsite

Conversation

@eseliger

@eseliger eseliger commented May 8, 2024

Copy link
Copy Markdown
Member

This has only one caller and is now implemented using existing gitserver stuff, so I'm moving this custom response mapping to where it's used. One less API we need to expose via gRPC.

Closes https://github.com/sourcegraph/sourcegraph/issues/62102

Test plan:

Adjusted tests, and existing E2E tests still pass.

@github-actions github-actions Bot added team/product-platform team/source Tickets under the purview of Source - the one Source to graph it all labels May 8, 2024
@eseliger eseliger force-pushed the es/05-08-gitserverimplementcommitsuniquetobranchusingcommits branch from 328eaa6 to 52ad1ff Compare May 8, 2024 00:53
@eseliger eseliger force-pushed the es/05-08-gitservermovecommitsuniquetobranchtocallsite branch from a23ce9f to ca98456 Compare May 8, 2024 00:53
@eseliger eseliger marked this pull request as ready for review May 8, 2024 15:45
@eseliger eseliger requested a review from a team May 8, 2024 15:46
@graphite-app

graphite-app Bot commented May 8, 2024

Copy link
Copy Markdown

Graphite Automations

"Assign Source to gitserver PRs" took an action on this PR • (05/08/24)

3 assignees were added to this PR based on Geoffrey Gilmore's automation.

Comment thread internal/codeintel/policies/matcher.go Outdated
Comment thread internal/codeintel/policies/matcher_common_test.go Outdated
@eseliger eseliger force-pushed the es/05-08-gitserverimplementcommitsuniquetobranchusingcommits branch from 52ad1ff to f1f50cb Compare May 10, 2024 18:38
@eseliger eseliger force-pushed the es/05-08-gitservermovecommitsuniquetobranchtocallsite branch from ca98456 to c5a4188 Compare May 10, 2024 18:39
@eseliger eseliger force-pushed the es/05-08-gitserverimplementcommitsuniquetobranchusingcommits branch from f1f50cb to 2bf43b1 Compare May 10, 2024 19:27
@eseliger eseliger force-pushed the es/05-08-gitservermovecommitsuniquetobranchtocallsite branch from c5a4188 to 7d955ac Compare May 10, 2024 19:27
@eseliger eseliger force-pushed the es/05-08-gitserverimplementcommitsuniquetobranchusingcommits branch from 2bf43b1 to bba7e86 Compare May 10, 2024 19:29
@eseliger eseliger force-pushed the es/05-08-gitservermovecommitsuniquetobranchtocallsite branch from 7d955ac to 99767c0 Compare May 10, 2024 19:29
@eseliger eseliger force-pushed the es/05-08-gitserverimplementcommitsuniquetobranchusingcommits branch from bba7e86 to 5cb7b4d Compare May 10, 2024 22:25
@eseliger eseliger force-pushed the es/05-08-gitservermovecommitsuniquetobranchtocallsite branch from 99767c0 to 4c79f9f Compare May 10, 2024 22:25
@eseliger eseliger force-pushed the es/05-08-gitserverimplementcommitsuniquetobranchusingcommits branch 2 times, most recently from e0513d4 to 0301fe4 Compare May 10, 2024 22:29
@eseliger eseliger force-pushed the es/05-08-gitservermovecommitsuniquetobranchtocallsite branch from d83a205 to e9df5e4 Compare May 10, 2024 22:30
@eseliger eseliger force-pushed the es/05-08-gitserverimplementcommitsuniquetobranchusingcommits branch from 0301fe4 to c74a757 Compare May 10, 2024 23:01
Base automatically changed from es/05-08-gitserverimplementcommitsuniquetobranchusingcommits to main May 10, 2024 23:07
This has only one caller and is now implemented using existing gitserver stuff, so I'm moving this custom response mapping to where it's used. One less API we need to expose via gRPC.

Test plan:

Adjusted tests, and existing E2E tests still pass.
@eseliger eseliger force-pushed the es/05-08-gitservermovecommitsuniquetobranchtocallsite branch from e9df5e4 to b130050 Compare May 10, 2024 23:08

eseliger commented May 10, 2024

Copy link
Copy Markdown
Member Author

Merge activity

  • May 10, 7:08 PM EDT: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.
  • May 10, 7:16 PM EDT: @eseliger merged this pull request with Graphite.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed team/product-platform team/source Tickets under the purview of Source - the one Source to graph it all

Projects

None yet

Development

Successfully merging this pull request may close these issues.

gitserver: Move CommitsUniqueToBranch to new gRPC pattern

3 participants