Skip to content

parser: convert Location to be an ES6 class#2233

Merged
IvanGoncharov merged 1 commit intographql:masterfrom
IvanGoncharov:pr_branch
Oct 20, 2019
Merged

parser: convert Location to be an ES6 class#2233
IvanGoncharov merged 1 commit intographql:masterfrom
IvanGoncharov:pr_branch

Conversation

@IvanGoncharov
Copy link
Copy Markdown
Member

No description provided.

@IvanGoncharov IvanGoncharov added the PR: breaking change 💥 implementation requires increase of "major" version number label Oct 20, 2019
@IvanGoncharov IvanGoncharov merged commit ec5fbb0 into graphql:master Oct 20, 2019
@IvanGoncharov IvanGoncharov deleted the pr_branch branch October 20, 2019 19:32
@mjmahone
Copy link
Copy Markdown
Contributor

Note on this: we forgot to update the exports, so it's impossible for someone to create a new Location now. We only export Location as a type.

IvanGoncharov added a commit to IvanGoncharov/graphql-js that referenced this pull request May 29, 2020
IvanGoncharov added a commit that referenced this pull request May 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: breaking change 💥 implementation requires increase of "major" version number

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants