graphql-js icon indicating copy to clipboard operation
graphql-js copied to clipboard

Keep order of specified definitions consistent with the spec

Open martinbonnin opened this issue 6 months ago • 1 comments

See https://github.com/graphql/graphql-spec/pull/1037#discussion_r2026533747

martinbonnin avatar Jun 05 '25 16:06 martinbonnin

For context: we want to add a new appendix to the GraphQL spec that covers the full introspection schema. We want that to be produced by printing the GraphQL.js introspection schema, but we also want it to line up with the fields and arguments defined in the introspection chapter, so field order matters. This PR better aligns the field order in GraphQL.js with the specification.

benjie avatar Jun 19 '25 10:06 benjie

Made a few polls and people are predominantly sorting fields semantically.

Screenshot 2025-06-20 at 15 36 14 Screenshot 2025-06-20 at 15 36 36 Screenshot 2025-06-20 at 15 36 48

I'll close this PR and update the spec text instead. @benjie if you have any preferred order or if you'd prefer to change the order in both graphql-js and the spec, now is the time!

martinbonnin avatar Jun 20 '25 13:06 martinbonnin