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

Move printer and parser dependency to graphql, move to [email protected] #1

Closed
wants to merge 176 commits into from

Conversation

jnwng
Copy link
Owner

@jnwng jnwng commented Mar 13, 2017

per graphql-tag#54, the printer and parser exports from graphql-tag have been deprecated, and we should be using the equivalent export from the graphql package itself.

on [email protected]:

=============================== FileSize summary ===============================
The total size of index.min.js is 90.7 kB.
The total gzipped size of index.min.js is 24.4 kB.
================================================================================

on [email protected]:

=============================== FileSize summary ===============================
The total size of index.min.js is 90.7 kB.
The total gzipped size of index.min.js is 24.4 kB.
================================================================================

TODO:

  • If this PR is a new feature, reference an issue where a consensus about the design was reached (not necessary for small changes)
  • Make sure all of the significant new logic is covered by tests
  • Rebase your changes on master so that they can be merged easily
  • Make sure all tests and linter rules pass
  • Update CHANGELOG.md with your change
  • Add your name and email to the AUTHORS file (optional)
  • If this was a change that affects the external API, update the docs and post a link to the PR in the discussion

once graphql-tag hits 2.0 i'll switch the git tag.

Dmitry Dushkin and others added 30 commits January 5, 2017 13:32
We need dispatch  APOLLO_MUTATION_ERROR before reject ApolloError. This error throw from server side with status code 2xx.

This patch fix optimistic response rollback issue apollographql#1386
then rc-2 has been introduced in  apollographql@f8e48d9, the wrong version string has slipped into the changelog. 
Fixing that...
One of these days, we should just stop the coverage from running on node 4.
…spect

Added benchmark:inspect to package.json
…initial

Update dependencies to enable Greenkeeper 🌴
@jnwng jnwng closed this Mar 28, 2017
@jnwng jnwng deleted the jw/use-graphql branch March 28, 2017 04:27
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.