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

gitserver: grpc: remove now unused DiffSymbols func#62360

Merged
ggilmore merged 1 commit intomainfrom
05-01-gitserver_grpc_remove_now_unused_diffsymbols_func
May 7, 2024
Merged

gitserver: grpc: remove now unused DiffSymbols func#62360
ggilmore merged 1 commit intomainfrom
05-01-gitserver_grpc_remove_now_unused_diffsymbols_func

Conversation

@ggilmore
Copy link
Contributor

@ggilmore ggilmore commented May 1, 2024

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

This PR removes the (now unused) DiffSymbols method from the gitserver client.

Test plan

CI

Copy link
Contributor Author

ggilmore commented May 1, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ggilmore and the rest of your teammates on Graphite Graphite

@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 1, 2024
@ggilmore ggilmore force-pushed the 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method branch from 85029be to 69e4bb9 Compare May 1, 2024 21:18
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from 9bba630 to 5427583 Compare May 1, 2024 21:18
@ggilmore ggilmore force-pushed the 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method branch from 69e4bb9 to aaf73ae Compare May 1, 2024 21:22
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from 5427583 to 45a925d Compare May 1, 2024 21:22
@ggilmore ggilmore force-pushed the 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method branch from aaf73ae to 2c110b4 Compare May 1, 2024 21:25
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from 45a925d to b09c7c7 Compare May 1, 2024 21:25
@ggilmore ggilmore force-pushed the 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method branch from 2c110b4 to 3d38d96 Compare May 1, 2024 21:30
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from b09c7c7 to 34f5ee4 Compare May 1, 2024 21:30
@ggilmore ggilmore force-pushed the 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method branch from 3d38d96 to 253343d Compare May 1, 2024 21:53
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from 34f5ee4 to 5ba94db Compare May 1, 2024 21:53
@ggilmore ggilmore force-pushed the 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method branch from 253343d to 20091a7 Compare May 1, 2024 21:59
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from 5ba94db to 37fc836 Compare May 1, 2024 21:59
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from 1a3a387 to 5fbe0c4 Compare May 6, 2024 19:20
@ggilmore ggilmore force-pushed the 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method branch from f63624e to 93a4bc2 Compare May 6, 2024 19:24
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from 5fbe0c4 to 6d60e25 Compare May 6, 2024 19:24
@ggilmore ggilmore force-pushed the 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method branch from 93a4bc2 to ffed11d Compare May 6, 2024 19:38
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from 6d60e25 to e1a3be0 Compare May 6, 2024 19:38
@graphite-app
Copy link

graphite-app bot commented May 6, 2024

TV gif. Timmy from Shaun the Sheep blinks and extends 2 thumbs up as a lopsided grin emerges on the side of his face. (Added via Giphy)

@ggilmore ggilmore force-pushed the 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method branch from ffed11d to a57255b Compare May 6, 2024 21:56
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from e1a3be0 to 470d785 Compare May 6, 2024 21:56
@ggilmore ggilmore force-pushed the 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method branch from a57255b to ee202a6 Compare May 6, 2024 23:28
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from 470d785 to fedae8e Compare May 6, 2024 23:28
@ggilmore ggilmore force-pushed the 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method branch from ee202a6 to b5751c2 Compare May 6, 2024 23:35
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from fedae8e to 3895516 Compare May 6, 2024 23:35
@ggilmore ggilmore force-pushed the 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method branch from b5751c2 to bfb8ca5 Compare May 6, 2024 23:50
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from 3895516 to ee069a0 Compare May 6, 2024 23:51
@ggilmore ggilmore force-pushed the 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method branch from bfb8ca5 to 9554328 Compare May 7, 2024 03:48
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from ee069a0 to 83da99f Compare May 7, 2024 03:48
@ggilmore ggilmore force-pushed the 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method branch from 9554328 to 95b8636 Compare May 7, 2024 16:13
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from 83da99f to 039b580 Compare May 7, 2024 16:14
@ggilmore ggilmore force-pushed the 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method branch from 95b8636 to 353136f Compare May 7, 2024 16:16
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from 039b580 to c5499b6 Compare May 7, 2024 16:16
@ggilmore ggilmore force-pushed the 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method branch from 353136f to 425cc8a Compare May 7, 2024 16:18
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from c5499b6 to b568cfc Compare May 7, 2024 16:18
@ggilmore ggilmore force-pushed the 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method branch from 425cc8a to c2000a7 Compare May 7, 2024 18:21
Base automatically changed from 05-01-worker_embeddings_replace_usage_of_old_gitserver.diffsymbols_client_method_with_new_gitserver.changedfiles_method to main May 7, 2024 18:33
@ggilmore ggilmore force-pushed the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch from b568cfc to 42d0d0c Compare May 7, 2024 18:33
Copy link
Contributor Author

ggilmore commented May 7, 2024

Merge activity

  • May 7, 2:34 PM EDT: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.
  • May 7, 2:50 PM EDT: @ggilmore merged this pull request with Graphite.

@ggilmore ggilmore merged commit b9cb4c5 into main May 7, 2024
@ggilmore ggilmore deleted the 05-01-gitserver_grpc_remove_now_unused_diffsymbols_func branch May 7, 2024 18:50
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.

Provide GQL API to list all files and change mode in a given diff

2 participants