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

Support for GraphQL IDL schema file #59

Closed
bessey opened this issue May 17, 2017 · 3 comments
Closed

Support for GraphQL IDL schema file #59

bessey opened this issue May 17, 2017 · 3 comments
Labels
enhancement feature New addition or enhancement to existing solutions help wanted

Comments

@bessey
Copy link

bessey commented May 17, 2017

Feature suggestion: the linter, unlike graphql-tool’s makeExecutableSchema, relies not on a GraphQL IDL file, but instead on the much longer introspection query file.

I version control my schema with each client, so there test suites can be run against a static target, it would be nice (tm) if I didn't need to do any legwork to convert my IDL to the JSON format.

@bessey
Copy link
Author

bessey commented May 17, 2017

Per @jnwng's reply on Slack:

i think the output of buildSchemaFromTypeDefinitions could be used to translate the IDL back to the schema we can parse with the graphql library (which we’re using under the hood)

@jnwng jnwng added enhancement feature New addition or enhancement to existing solutions labels May 24, 2017
@jnwng
Copy link
Contributor

jnwng commented May 24, 2017

definitely, i like this since we can generate the executable schema. would happily accept a PR to this effect!

@stubailo
Copy link
Contributor

Context: When this tool was first introduced, the IDL actually didn't exist! So adding it as an option alongside the introspection result would be awesome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement feature New addition or enhancement to existing solutions help wanted
Projects
None yet
Development

No branches or pull requests

3 participants