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
Get arguments with directive #4661
Conversation
🦋 Changeset detectedLatest commit: 96d5e26 The changes in this PR will be included in the next version bump. This PR includes changesets to release 28 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 |
@nicolaslt is attempting to deploy a commit to the The Guild Team on Vercel. A member of the Team first needs to authorize it. |
Hmm the test suite seems to be unstable right now. I think I'll wait for a review before trying to get all-green (it seems everything that fails is unrelated to my changes) |
It is ok. We can merge it. Thanks @nicolaslt :) |
Description
Adds a new helper function proposed in #4626.
I have also removed replaced the custom AST value parsing code with the official helper from
graphql-js
. It seems to be available all the way to the minimum supported peerDependency version. See this one for version v14.5.6Related #4626
Type of change
Please delete options that are not relevant.
Screenshots/Sandbox (if appropriate/relevant):
N/A
How Has This Been Tested?
Only relied on the unit tests (old and new)
Test Environment:
@graphql-tools/...
: base of my branchChecklist: