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

fix: update @graphql-typed-document-node/core for graphql@16 support #156

Merged
merged 4 commits into from
Feb 15, 2022

Conversation

SimenB
Copy link
Contributor

@SimenB SimenB commented Feb 4, 2022

Not sure why the version is locked down, but 3.1.0 does not have graphql@16 in its peer dependency range

@mcollina
Copy link

mcollina commented Feb 4, 2022

This might explains I few of the problems I hit, good spot @SimenB!

@@ -81,7 +81,7 @@
"typescript": "^4.4.4"
},
"dependencies": {
"@graphql-typed-document-node/core": "3.1.0",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was contributed in #139

@ardatan do you recall any special reason why we would have locked the version to 3.1.0?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, ideally it'd just be ^3.1.0 I guess 🙂

@oporkka
Copy link
Member

oporkka commented Feb 4, 2022

@SimenB please run also yarn install to update the lockfile

@oporkka oporkka closed this Feb 4, 2022
@oporkka oporkka reopened this Feb 4, 2022
@oporkka
Copy link
Member

oporkka commented Feb 4, 2022

Sorry for accidental close

package.json Outdated Show resolved Hide resolved
SimenB and others added 2 commits February 4, 2022 10:40
@ruiaraujo
Copy link
Collaborator

👍

1 similar comment
@oporkka
Copy link
Member

oporkka commented Feb 15, 2022

👍

@oporkka oporkka merged commit 0dac810 into zalando-incubator:main Feb 15, 2022
@SimenB SimenB deleted the patch-1 branch February 15, 2022 11:28
@SimenB
Copy link
Contributor Author

SimenB commented Feb 15, 2022

Thanks! Could you release this as well? 😀

@ruiaraujo
Copy link
Collaborator

ruiaraujo commented Feb 15, 2022

@SimenB going out now, thanks for the contribution.

@SimenB
Copy link
Contributor Author

SimenB commented Feb 15, 2022

Awesome, thanks!

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

Successfully merging this pull request may close these issues.

4 participants