search

graphql-rust / graphql-parser

A graphql query language and schema definition language parser and formatter for rust
Apache License 2.0
355 stars 77 forks source link

Add VARIABLE_DEFINITION directive location #66

Closed vmagro closed 1 year ago

vmagro commented 2 years ago

I have some schemas that have this directive location, and this crate currently crashes while trying to parse it.

LegNeato commented 2 years ago

Perhaps this and https://github.com/graphql-rust/graphql-parser/pull/61 can be merged

vmagro commented 2 years ago

@LegNeato I added a test that is similar to other directive parsers, let me know if you had something else in mind

bolinfest commented 2 years ago

@LegNeato once this lands, any chance we could get a new version of this crate pushed to crates.io along with a new version of graphql-client to pick up https://github.com/graphql-rust/graphql-client/pull/430?

dotansimha commented 9 months ago

@LegNeato I think this PR was never included in a release? any chance we can get a release for that fix?