-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
LSP Rewrite Step 2: refactor for @graphql-tools
#3556
base: main
Are you sure you want to change the base?
Conversation
|
@graphql-tools
@graphql-tools
a608668
to
7839f79
Compare
e8654b6
to
76c0d91
Compare
f24bb65
to
ea57543
Compare
0496caa
to
203a2c0
Compare
fb7d418
to
e5ab31e
Compare
this gets us most of the way, just one integration suite assertion failing and some ts bugs. it needs a lot more work to be ready. the preceeding PR happens first of course
one of the end goals of this PR is also network schema invalidation :)) yes that's right, code first for all!
this also sets us up nicely for adding multi-root workspaces support, as we are reducing CPU and memory usage heavily in this PR, and this also provides a cleaner interface for MessageProcessor to support it
TODO: