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

Disallow non-breakable chains of circular references in Input Objects #701

Merged

Conversation

spawnia
Copy link
Member

@spawnia spawnia commented Mar 24, 2020

@leebyron I accidentally nuked the previous PR #445 by adding the Pull Probot, which hard-reset my master branch. Sorry!

@IvanGoncharov
Copy link
Member

@spawnia No problem. Thanks for opening the new PR.

@IvanGoncharov
Copy link
Member

@spawnia Thinking about this more. Can you force push these changes to master and I will reopen #445?

@spawnia
Copy link
Member Author

spawnia commented Mar 26, 2020

@spawnia Thinking about this more. Can you force push these changes to master and I will reopen #445?

Pull Probot will just nuke my branch again when i do that.

lirsacc added a commit to lirsacc/py-gql that referenced this pull request Nov 1, 2020
This is based on a spec proposal (see:
graphql/graphql-spec#701) and may change in the
future.
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jan 11, 2021

CLA Not Signed

@leebyron
Copy link
Collaborator

@spawnia I'm in the process of finalizing editorial review before publishing the next spec edition and merging in accepted RFCs in the process.

I'm merging this now, but if you wouldn't mind please verifying that you've signed the EasyCLA document above!

@leebyron leebyron merged commit eb86ed9 into graphql:master Jan 11, 2021
@spawnia
Copy link
Member Author

spawnia commented Jan 11, 2021

@leebyron Thank you, i just signed. This must have been the third or fourth time i have had to do this - whatever it takes 🤷

@spawnia spawnia deleted the disallow-impossible-input-object-cycles branch January 11, 2021 13:34
@leebyron
Copy link
Collaborator

leebyron commented Jan 11, 2021 via email

@leebyron leebyron added this to the May2021 milestone Apr 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏁 Accepted (RFC 3) RFC Stage 3 (See CONTRIBUTING.md)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants