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

Emit descriptions as block strings in printer.Print #485

Merged
merged 5 commits into from
Apr 26, 2020

Conversation

mmmveggies
Copy link
Contributor

Fixes #484

Adds description printing to the printer.Print method, with tests to verify.
Seeks to keep field/directive arguments inline if none among all siblings have a description.

@coveralls
Copy link

coveralls commented Jun 4, 2019

Coverage Status

Coverage remained the same at 92.376% when pulling 4339edf on mmmveggies:print-comments into 02caa89 on graphql-go:master.

@mmmveggies
Copy link
Contributor Author

@bhoriuchi are y'all waiting for a release...?

@bhoriuchi
Copy link
Contributor

bhoriuchi commented Jul 19, 2019

@mmmveggies I don’t have write access on this project to my knowledge. Just a contributor like you.

@mikekaeding
Copy link

Does anyone know if this fix will get added into the code soon?

@negararazm
Copy link

Anyone with the write access?

@chris-ramon chris-ramon self-requested a review April 24, 2020 20:59
Copy link
Member

@chris-ramon chris-ramon left a comment

Choose a reason for hiding this comment

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

LGTM 👍 –– Thanks a lot @mmmveggies 💯 🚢

@chris-ramon chris-ramon merged commit 116f19d into graphql-go:master Apr 26, 2020
@mmmveggies mmmveggies deleted the print-comments branch May 28, 2020 16:28
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.

printer package Print does not print descriptions
6 participants