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

Use graphql v14.0.0 #3868

Closed
wants to merge 3 commits into from
Closed

Use graphql v14.0.0 #3868

wants to merge 3 commits into from

Conversation

kamilkisiela
Copy link
Contributor

@kamilkisiela kamilkisiela commented Aug 31, 2018

I removed packages/*/package-lock.json because it's not something lerna produce and it's just trash. Someone just ran npm install in subdirectories and that's why it got here.

apollo-cache-inmemory had no ^14.0.0 in peerDeps.

Related PR: apollographql/apollo-link#777
Apollo Angular works with new graphql too: kamilkisiela/apollo-angular#795

@hwillson one question, have you guys changed renovate's configuration to not create PRs? I see there's a branch available.

@@ -45,7 +45,7 @@
"graphql-anywhere": "file:../graphql-anywhere"
},
"peerDependencies": {
"graphql": "0.11.7 || ^0.12.0 || ^0.13.0"
"graphql": "0.11.7 || ^0.12.0 || ^0.13.0 || ^14.0.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

"0.11.7 - 14" is more succinct and easier to update.

@raja
Copy link

raja commented Oct 2, 2018

What is that status of this PR ?

@daggerjames
Copy link

duplicated with #3978

@hwillson
Copy link
Member

This was addressed in b082c8b. Thanks @kamilkisiela!

@hwillson hwillson closed this Dec 12, 2018
@hwillson hwillson deleted the update/graphql-14 branch December 12, 2018 19:09
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants