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

codeintel: Add stub implementation for usagesForSymbol API#62926

Merged
varungandhi-src merged 8 commits into
mainfrom
vg/usagesForSymbol
May 29, 2024
Merged

codeintel: Add stub implementation for usagesForSymbol API#62926
varungandhi-src merged 8 commits into
mainfrom
vg/usagesForSymbol

Conversation

@varungandhi-src

@varungandhi-src varungandhi-src commented May 27, 2024

Copy link
Copy Markdown
Contributor

Let's merge a stub implementation with the types in place.

This will allow parallelizing the spike on syntactic and the work
on implementing the API for precise.

Test plan

Ran locally - got Not implemented yet error.

CleanShot 2024-05-27 at 19 44 58@2x

@cla-bot cla-bot Bot added the cla-signed label May 27, 2024
@github-actions github-actions Bot added team/graph Graph Team (previously Code Intel/Language Tools/Language Platform) team/product-platform labels May 27, 2024
Comment thread internal/codeintel/codenav/transport/graphql/root_resolver.go Outdated
Comment thread internal/codeintel/codenav/transport/graphql/root_resolver.go Outdated
Comment thread cmd/frontend/graphqlbackend/git_tree_entry.go Outdated
Comment thread cmd/frontend/graphqlbackend/codeintel.codenav.graphql Outdated

@kritzcreek kritzcreek 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.

LGTM as scaffolding

Comment thread internal/codeintel/codenav/transport/graphql/root_resolver.go Outdated
@varungandhi-src varungandhi-src enabled auto-merge (squash) May 28, 2024 14:15
@varungandhi-src varungandhi-src merged commit b61ef32 into main May 29, 2024
@varungandhi-src varungandhi-src deleted the vg/usagesForSymbol branch May 29, 2024 10:02
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