Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add SchemaDocument go to definition LSP support #4669

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tobias-tengler
Copy link
Contributor

@tobias-tengler tobias-tengler commented Apr 5, 2024

Adds go to definition support for types and directives within a .graphql file.

@tobias-tengler tobias-tengler marked this pull request as ready for review April 5, 2024 20:48
@tobias-tengler tobias-tengler force-pushed the lsp-schema-go-to-definition branch 2 times, most recently from a09adc0 to 6c1159b Compare April 10, 2024 17:57
@tobias-tengler
Copy link
Contributor Author

tobias-tengler commented May 9, 2024

@captbaritone Thanks for merging the SchemaDocument support :) This is now also ready to be merged.

EDIT: Remember that you need to run the VS Code extension from source when testing. Otherwise the LSP will not be activated for .graphql files. Or alternatively, release a new version of the extension. I've already implemented support and bumped the version in 964e078.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants