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

Update graphql to the latest version 🚀 #67

Merged
merged 4 commits into from
May 31, 2017
Merged

Conversation

greenkeeper[bot]
Copy link

@greenkeeper greenkeeper bot commented May 25, 2017

Version 0.10.0 of graphql just got published.

Dependency graphql
Current Version 0.9.6
Type dependency

The version 0.10.0 is not covered by your current version range.

Without accepting this pull request your project will work just like it did before. There might be a bunch of new features, fixes and perf improvements that the maintainers worked on for you though.

I recommend you look into these changes and try to get onto the latest version of graphql.
Given that you have a decent test suite, a passing build is a strong indicator that you can take advantage of these changes by merging the proposed change into your project. Otherwise this branch is a great starting point for you to work on the update.


Release Notes v0.10.0

New

  • Initial support for GraphQL subscriptions (#846).
  • isValidValue() and isValidLiteral() added to Scalar and enum types (#861).
  • You can now set a custom default field resolver (#865).
  • graphql() may now take a Source object, in addition to the existing support for string (#866).
  • graphql() may now take an object with named parameters (#867).

Breaking

  • Spec compliance: forbid duplicate type definitions (#744).
  • Spec compliance: disallow inappropriate coercion of non-Int values to Int (#837).

Fixes

  • Ensure GraphQLError stack trace includes error message (#718).
  • Fix input coercion to allow null-valued Enums as arguments (#848).
  • Fix for fields that are JavaScript keywords (#864).
  • Compatibility with Flow v0.47.0 (#878).
Commits

The new version differs by 57 commits.

  • 797e7e3 0.10.0
  • 845a45b Merge pull request #878 from gabelevi/flow
  • fe5f544 Flow v0.47.0 finds 3 errors in master. This is important to fix, because the
  • 87b24ee Freshen yarn.lock
  • 704a97d Merge pull request #877 from graphql/greenkeeper/mocha-3.4.2
  • 0f186de chore(package): update mocha to version 3.4.2
  • 1ff4932 Freshen yarn.lock
  • e989de8 Merge pull request #875 from graphql/greenkeeper/sane-1.7.0
  • 9811d06 chore(package): update sane to version 1.7.0
  • f6f26fd Handle abrupt closing in mapAsyncIterator() (#870)
  • 50504b9 remove unnecessary null arguments in test
  • 562cead Allow providing an object of arguments to graphql(). (#867)
  • 1b41959 Allow graphql() to accept a Source instance. (#866)
  • bf352a8 json is hard
  • 5510c75 Allow providing custom default field resolver. (#865)

There are 57 commits in total.

See the full diff

Not sure how things should work exactly?

There is a collection of frequently asked questions and of course you may always ask my humans.


Your Greenkeeper Bot 🌴

@stubailo
Copy link
Contributor

Looks like the error message thrown changed in 0.10 for no reason, unfortunate!

@stubailo stubailo requested a review from jnwng May 29, 2017 04:06
@stubailo
Copy link
Contributor

@jnwng I've fixed the tests, feel free to merge and publish whenever. It looks like you bumped the version last time when GraphQL.js was upgraded.

Also, might it be worth it to jump to 1.0.0?

@jnwng
Copy link
Contributor

jnwng commented May 30, 2017

definitely. will merge and bump to 1.0 on wednesday.

@Urigo
Copy link

Urigo commented May 30, 2017

@jnwng maybe we should bump graphql to 0.10.1.. they changed something in how they handle errors and that makes a difference in some tests

@jnwng jnwng force-pushed the greenkeeper/graphql-0.10.0 branch from ed04b21 to 38e63ef Compare May 30, 2017 22:02
@jnwng jnwng merged commit 86f822e into master May 31, 2017
@jnwng jnwng deleted the greenkeeper/graphql-0.10.0 branch May 31, 2017 18:56
jtschulz pushed a commit to jtschulz/eslint-plugin-graphql that referenced this pull request Jul 12, 2017
* fix(package): update graphql to version 0.10.0

* Fix tests

* Bumping to `graphql@0.10.1`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants