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

Avoid extensions override #3342

Conversation

SergioCrisostomo
Copy link

In 15.7.1 there is a bug (not present in the 16 branch) that when an original error is passed and has extensions the argument extensions passed to the constructor is ignored.

This PR fixes it and aims to be a patch release, maybe 15.7.2. Not sure how your prefer to do that so happy to get some guidance if needed anything more from me.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Oct 27, 2021

CLA Missing ID

@@ -1,6 +1,6 @@
{
"name": "graphql",
"version": "15.7.1",
"version": "15.7.2",
Copy link
Member

Choose a reason for hiding this comment

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

Please revert version bump, this is done in a separate flow.

@@ -6,14 +6,14 @@
/**
* A string containing the version of the GraphQL.js library
*/
export const version = '15.7.1';
export const version = '15.7.2';
Copy link
Member

Choose a reason for hiding this comment

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

revert please :)

@SergioCrisostomo
Copy link
Author

Closing in favour of #3343

@IvanGoncharov
Copy link
Member

@SergioCrisostomo Thanks for PR 👍

Released as https://github.com/graphql/graphql-js/releases/tag/v15.7.2

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.

None yet

3 participants