fix: be sure to register directive def at source scope in grammar

graphql / vscode-graphql

MIGRATED: VSCode GraphQL extension (autocompletion, go-to definition, syntax highlighting)

https://marketplace.visualstudio.com/items?itemName=Prisma.vscode-graphql

MIT License

555 stars 71 forks source link

fix: be sure to register directive def at source scope in grammar #188

Closed acao closed 4 years ago

acao commented 4 years ago

in #183 we forgot this!

Before, we had just fixed inline argument comments for directive definitions and other types with inline arguments, but we had not solved the top-level keyword/etc definition issue. graphql 15 Directive definitions before #183 were still almost all green, unhighlighted!

Before (after #183):

After: