GraphQL: SDL Support about swift-graphql

[maticzav]

@Sundea thank you for reporting this, but this is not a bug or at least I wouldn't consider it.

The thing is that the generating works out of the JSON dump of the schema, but not yet out of the GraphQL DSL schema. Is this a blocker for you? I think it's not completely trivial to parse the schema DSL into the format that the generator expects, but I'd happily help you give it a stab 🙂

[vladimirlogachev]

Most GraphQL code generators work with SDL, so it's sort of expected.
But anyway, thank you for the explanation.
And for the library itself, too! 🤎

[Fedos]

I'd double that I also expected from Codegen to support SDL schema as an input, but I see that it's not trivial, so in the meantime I use this online SDL > JSON converter https://transform.tools/graphql-to-introspection-json , the only thing missing in it's output is that you have to nest it in the "data" key.

BTW, @maticzav , would be nice if the integrations tests of the codegen used the schema from the file instead of using localhost server (which I haven't figured out how to run)
Here is the converted schema file for that:
schema.json.zip

[pokryfka]

BTW, @maticzav , would be nice if the integrations tests of the codegen used the schema from the file instead of using localhost server (which I haven't figured out how to run) Here is the converted schema file for that: schema.json.zip

I echo that.

[shaps80]

BTW, @maticzav , would be nice if the integrations tests of the codegen used the schema from the file instead of using localhost server (which I haven't figured out how to run) Here is the converted schema file for that: schema.json.zip

I echo that.

💯

I'll make a new Issue to keep track of this 👍

[shaps80]

After further discussion @maticzav and I have decided that this is fairly low priority given we're making a library for App users which will 99% of the time already have access to a GraphQL schema and as such can use introspection.

That being said, we have substantial plans in v6 to reduce code-generation times to seconds in almost all cases, so this may be less of an issue in the coming days.

We will re-review if and when further performance enhancements need to be met.