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

Add a prefer_single_line_descriptions option on SDLExportOptions #955

Merged
merged 2 commits into from Jun 25, 2022
Merged

Add a prefer_single_line_descriptions option on SDLExportOptions #955

merged 2 commits into from Jun 25, 2022

Conversation

ivan
Copy link
Contributor

@ivan ivan commented Jun 20, 2022

The default export uses three lines for each description. Single-line descriptions can improve the readability of the exported SDL when there are many short descriptions.

With prefer_single_line_descriptions , the exported output looks like
https://gist.github.com/ivan/f8e394651a86a016a7d99258fbc556ff
instead of
https://gist.github.com/ivan/9b0e473e4a2a600d93f1b1610a40260a

Please let me know what you think, I am happy to change things if you want.

ivan added 2 commits June 20, 2022 10:26
The default export uses three lines for each description. Single-line
descriptions improve the readability of the exported SDL when there are
many short descriptions.
I confirmed that graphql-code-generator properly handles SDL with this escaping.
@sunli829 sunli829 merged commit 2d79376 into async-graphql:master Jun 25, 2022
@sunli829
Copy link
Collaborator

Thanks! 🙂

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.

None yet

2 participants