Geoffrey Gilmore comments

Results 41 comments of


                                            Geoffrey Gilmore

searcher: remove unsued diff.ParseGitDiffNameStatus package

### Merge activity * **May 7, 2:51 PM EDT**: [Graphite](https://app.graphite.dev/github/pr/sourcegraph/sourcegraph/62363) rebased this pull request as part of a merge. * **May 7, 3:01 PM EDT**: @ggilmore merged this pull request...

gitserver: grpc: remove now unused DiffSymbols func

* **#62363** * **#62360** 👈 * **#62359** * **#62358** * **#62355** * **#62354** * **#62262** * **#62252** * **#62217** * **#62216** * **#62212** * **#62302** * **#62173** * **#62169** *...

gitserver: grpc: remove now unused DiffSymbols func

### Merge activity * **May 7, 2:34 PM EDT**: [Graphite](https://app.graphite.dev/github/pr/sourcegraph/sourcegraph/62360) rebased this pull request after merging its parent, because this pull request is set to merge when ready. * **May...

worker: embeddings: replace usage of old gitserver.DiffSymbols client method with new gitserver.ChangedFiles method

* **#62363** * **#62360** * **#62359** 👈 * **#62358** * **#62355** * **#62354** * **#62262** * **#62252** * **#62217** * **#62216** * **#62212** * **#62302** * **#62173** * **#62169** *...

worker: embeddings: replace usage of old gitserver.DiffSymbols client method with new gitserver.ChangedFiles method

### Merge activity * **May 7, 2:22 PM EDT**: [Graphite](https://app.graphite.dev/github/pr/sourcegraph/sourcegraph/62359) rebased this pull request as part of a merge. * **May 7, 2:33 PM EDT**: @ggilmore merged this pull request...

searcher: replace usage of gitserver.DiffSymbols with new gitserver.ChangedFiles client method

* **#62363** * **#62360** * **#62359** * **#62358** 👈 * **#62355** * **#62354** * **#62262** * **#62252** * **#62217** * **#62216** * **#62212** * **#62302** * **#62173** * **#62169** *...

searcher: replace usage of gitserver.DiffSymbols with new gitserver.ChangedFiles client method

### Merge activity * **May 7, 2:10 PM EDT**: [Graphite](https://app.graphite.dev/github/pr/sourcegraph/sourcegraph/62358) rebased this pull request as part of a merge. * **May 7, 2:20 PM EDT**: @ggilmore merged this pull request...

gitserver: grpc: add new ChangedFiles method to gitserver client

* **#62363** * **#62360** * **#62359** * **#62358** * **#62355** * **#62354** 👈 * **#62262** * **#62252** * **#62217** * **#62216** * **#62212** * **#62302** * **#62173** * **#62169** *...

gitserver: grpc: add new ChangedFiles method to gitserver client

@sourcegraph/source Can you please take another look at this PR (and all the upstack PRs that use it)? I refactored the approach here to use iterators.

gitserver: grpc: add new ChangedFiles method to gitserver client

### Merge activity * **May 7, 1:45 PM EDT**: [Graphite](https://app.graphite.dev/github/pr/sourcegraph/sourcegraph/62354) rebased this pull request after merging its parent, because this pull request is set to merge when ready. * **May...