-
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
move GraphQL utils from graphiql
to @graphiql/toolkit
#2419
Conversation
🦋 Changeset detectedLatest commit: e4d01cb The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Codecov Report
@@ Coverage Diff @@
## main #2419 +/- ##
==========================================
+ Coverage 65.70% 69.02% +3.31%
==========================================
Files 85 72 -13
Lines 5106 4587 -519
Branches 1631 1517 -114
==========================================
- Hits 3355 3166 -189
+ Misses 1747 1417 -330
Partials 4 4
Continue to review full report at Codecov.
|
090f48c
to
e5551ba
Compare
e817f50
to
903dbb7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
e5551ba
to
8000ace
Compare
903dbb7
to
52afc40
Compare
The latest changes of this PR are not available as canary, since there are no linked |
We move over:
fillLeafs
getSelectedOperationName
mergeAst
Also the
getQueryFacts
utility module was unused (just re-exporting stuff fromgraphql-language-service
), so we remove it.