Skip to content

Conversation

@JoviDeCroock
Copy link
Contributor

Implements the new Executable document descriptions as specified in graphql/graphql-spec#1170

@JoviDeCroock JoviDeCroock force-pushed the fragment-descriptions branch 2 times, most recently from b7676bd to c56045d Compare July 3, 2025 15:05
Copy link
Contributor

@adampetro adampetro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

Minor nit: can you add newlines to the ends of the test_data files?

@JoviDeCroock JoviDeCroock force-pushed the fragment-descriptions branch from c56045d to 7df7605 Compare July 3, 2025 15:55
Copy link
Contributor

@swalkinshaw swalkinshaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 TIL about this new spec feature

@JoviDeCroock JoviDeCroock merged commit 83b54d8 into main Jul 3, 2025
4 checks passed
@JoviDeCroock JoviDeCroock deleted the fragment-descriptions branch July 3, 2025 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants