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 defaultFieldResolver from graphql-js package instead of own one #373

Merged

Conversation

dimitri-koenig
Copy link
Contributor

Use of defaultFieldResolver of graphql-js package instead own one.
Should fix #368 as soon as this PR is merged: graphql/graphql-js#980

@dimitri-koenig
Copy link
Contributor Author

@stubailo: do you mind merging this PR though some minor coverage decrease has been detected?

@stubailo
Copy link
Contributor

Sure thing! Weird that it failed the status with that small of a change.

@stubailo stubailo merged commit 3aafbfe into ardatan:master Aug 14, 2017
@stubailo
Copy link
Contributor

@dimitri-koenig do you mind sending a PR to the changelog as well?

@dimitri-koenig dimitri-koenig deleted the feature/defaultFieldResolver branch August 14, 2017 16:56
@dimitri-koenig dimitri-koenig restored the feature/defaultFieldResolver branch August 14, 2017 16:56
dimitri-koenig added a commit to dimitri-koenig/graphql-tools that referenced this pull request Aug 14, 2017
@dimitri-koenig
Copy link
Contributor Author

dimitri-koenig commented Aug 14, 2017

@stubailo: Done. Please merge again. Any idea about a date of the next official release?

// Edit: need another standalone PR

dimitri-koenig added a commit to dimitri-koenig/graphql-tools that referenced this pull request Aug 14, 2017
stubailo pushed a commit that referenced this pull request Aug 14, 2017
@stubailo
Copy link
Contributor

@martijnwalraven or @freiksenet can publish!

@martijnwalraven
Copy link
Contributor

@dimitri-koenig: Just published v1.2.0!

@dimitri-koenig
Copy link
Contributor Author

@martijnwalraven & @stubailo: you are awesome. That way open source work is really so much fun. Efficient, effective and fast. Thank you so much.

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.

3 participants