-
-
Notifications
You must be signed in to change notification settings - Fork 227
Upgrade Graphql-js version #55
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
Conversation
… is not executable
@amian84 Thanks for PR 👍 |
I added more changes fixing when you extend another fake schema. If you try extend another faker schema, the buildSchema function failed because the system defines the faker schema more than once |
…the version 0.9.3 new version of graphql
Hey here ? Any news on this PR ? |
What about now? Any news? |
Any news? thanks |
guys, while accept PR you can clone my repository and checkout to 'distFolder' branch, then you can execute 'sudo npm link' and with that your global graphql-faker binary will be my build with the changes. |
@amian84 Thanks a lot for PR 👍 |
Hi @IvanGoncharov, I did a upgrade of graphql-js version and modify some code for work correclty.
With that, now the new description syntaxt are working but I don't know if all work propertly because for our propuse only need that (#41 ).
Also i had to disable the lint into codemirror because with the update, if you define any type on the schema always show an error about the type definition, the error is becuase is not executable... Maybe you can fix that.
I do the pull-request only for if you would like start your migration to new version with this base.
Thanks for all