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

codeintel: allows batch retrieval of SCIP documents#64024

Merged
kritzcreek merged 2 commits into
mainfrom
christoph/scip-documents-batch
Jul 25, 2024
Merged

codeintel: allows batch retrieval of SCIP documents#64024
kritzcreek merged 2 commits into
mainfrom
christoph/scip-documents-batch

Conversation

@kritzcreek

Copy link
Copy Markdown
Contributor

Closes https://linear.app/sourcegraph/issue/GRAPH-766/batch-api-for-fetching-scip-documents

Test plan

Tested by being used for the single-path retrieval path, will be tested more once its used in follow-up PRs with more batching

@kritzcreek kritzcreek requested a review from keynmol July 24, 2024 03:38
@cla-bot cla-bot Bot added the cla-signed label Jul 24, 2024
@github-actions github-actions Bot added team/graph Graph Team (previously Code Intel/Language Tools/Language Platform) team/product-platform labels Jul 24, 2024
@kritzcreek kritzcreek force-pushed the christoph/scip-documents-batch branch from 88f8cd4 to 4ea51bb Compare July 24, 2024 03:39
@kritzcreek kritzcreek force-pushed the christoph/scip-documents-batch branch from 4ea51bb to c90986c Compare July 24, 2024 12:02
Comment thread internal/codeintel/codenav/internal/lsifstore/lsifstore_documents.go Outdated

@keynmol keynmol left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ready to approve, just want some clarification on error possibly not being surfaced from scanner.

@kritzcreek kritzcreek force-pushed the christoph/scip-documents-batch branch from 1d65b42 to a706313 Compare July 25, 2024 07:47
@kritzcreek kritzcreek enabled auto-merge (squash) July 25, 2024 07:48
@kritzcreek kritzcreek requested a review from keynmol July 25, 2024 07:50
@kritzcreek kritzcreek merged commit f7688ff into main Jul 25, 2024
@kritzcreek kritzcreek deleted the christoph/scip-documents-batch branch July 25, 2024 08:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed team/graph Graph Team (previously Code Intel/Language Tools/Language Platform) team/product-platform

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants