Skip to content
This repository has been archived by the owner on May 16, 2019. It is now read-only.

Fixed required arrays when parsing operation variables #54

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jeggy
Copy link
Contributor

@jeggy jeggy commented Jan 31, 2019

Defining variables that either have an array or required array, didn't get parsed correctly. This fixes both $ids: [Int] and $ids: [Int!]!

@coveralls
Copy link

coveralls commented Jan 31, 2019

Coverage Status

Coverage increased (+0.2%) to 88.095% when pulling 1c62506 on jeggy:fix-required-variable-arrays into 42fef4a on pgutkowski:master.

@jeggy jeggy force-pushed the fix-required-variable-arrays branch from 9ef34db to cea39bc Compare February 2, 2019 13:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants