Skip to content
This repository was archived by the owner on Jun 29, 2021. It is now read-only.

Upgrade graphql to 14.6.0#104

Merged
vsumner merged 1 commit intomasterfrom
upgarde-graphql
Apr 13, 2020
Merged

Upgrade graphql to 14.6.0#104
vsumner merged 1 commit intomasterfrom
upgarde-graphql

Conversation

@vsumner
Copy link
Contributor

@vsumner vsumner commented Apr 8, 2020

Upgrades graphql from 14.2.0 to 14.6.0 and removes @types/graphql because graphql exports its own types since 14.5.0.

so many 🐛 fixes: 14.6.0

Some nice new features 14.5.0

  • Add 'extensions' to all Type System objects

More new features 14.4.0

  • Use 'Object.freeze' consistently on all exported Array/Object constants

This PR also updates apollo-codegen-core from 0.35.0 to 0.36.4 so that the same version of graphql can be used. https://github.com/apollographql/apollo-tooling/blob/master/CHANGELOG.md

@vsumner vsumner requested review from a team and patsissons April 8, 2020 21:19
Copy link
Contributor

@alexandcote alexandcote left a comment

Choose a reason for hiding this comment

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

This PR looks good to me. Small comment otherwise 🚢

@vsumner
Copy link
Contributor Author

vsumner commented Apr 9, 2020

Refactored the require "graphql": ">=14.5.0 <15.0.0"

Copy link
Contributor

@alexandcote alexandcote left a comment

Choose a reason for hiding this comment

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

All this make sense to me 👍

@vsumner vsumner merged commit 38c5686 into master Apr 13, 2020
@vsumner vsumner deleted the upgarde-graphql branch April 13, 2020 13:00
@vsumner vsumner deployed to production April 14, 2020 15:52 Active
@lemonmade lemonmade deployed to production June 15, 2020 17:04 Active
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.

3 participants