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

Remove unused `immutable` prop from ApolloProvider #539

Merged
merged 4 commits into from Apr 29, 2017

Conversation

Projects
None yet
4 participants
@mdebbar
Copy link
Contributor

mdebbar commented Mar 18, 2017

No description provided.

@meteor-bot

This comment has been minimized.

Copy link

meteor-bot commented Mar 18, 2017

@mdebbar: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Meteor Contributor Agreement here: https://contribute.meteor.com/

@thebigredgeek

This comment has been minimized.

Copy link
Contributor

thebigredgeek commented Apr 25, 2017

@mdebbar can you update the tests so that the build passes?

thebigredgeek and others added some commits Apr 25, 2017

@mdebbar

This comment has been minimized.

Copy link
Contributor Author

mdebbar commented Apr 26, 2017

@thebigredgeek I think the failure you saw was intermittent. I did Update branch again and tests are passing now.

James Baxley
@jbaxleyiii

This comment has been minimized.

Copy link
Member

jbaxleyiii commented Apr 29, 2017

Immutable support never did catch on did it haha. Thanks for cleaning this up!

@jbaxleyiii jbaxleyiii merged commit 56e0b1b into apollographql:master Apr 29, 2017

3 checks passed

CLA Author has signed the Meteor CLA.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 92.308%
Details

@jbaxleyiii jbaxleyiii referenced this pull request May 1, 2017

Merged

Release 1.1.3 #676

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment