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

Schema lang descriptions #464

Merged
merged 1 commit into from
Aug 24, 2016
Merged

Schema lang descriptions #464

merged 1 commit into from
Aug 24, 2016

Conversation

leebyron
Copy link
Contributor

Adds descriptions to the schema language via docblocks

This adds descriptions to the buildASTSchema (string->schema) and schemaPrinter (schema->string) via walking the previous full-line contiguous comment block. This is dependent on #463.

NOTE: This is a stacked diff - only changes to src/utilities belong to this PR

enum __TypeKind {

Copy link
Contributor

Choose a reason for hiding this comment

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

This newline is slightly odd, if I was writing this be hand I'd omit the initial newline.

@stubailo
Copy link

This is awesome! It will make writing schemas with the type language way better, descriptions were one of the biggest missing pieces.

@leebyron leebyron force-pushed the schema-lang-descriptions branch 2 times, most recently from ecdd26a to 654c48a Compare August 16, 2016 06:07
@leebyron leebyron changed the base branch from master to tokensss August 16, 2016 06:17
@leebyron
Copy link
Contributor Author

OMG stacked diffs work in github now.

@leebyron leebyron force-pushed the schema-lang-descriptions branch from 654c48a to 32c66a5 Compare August 16, 2016 06:18
@leebyron
Copy link
Contributor Author

@dschafer - I updated to newline before comment except when it's the first comment in a block/list

@leebyron leebyron changed the base branch from tokensss to master August 16, 2016 06:21
@leebyron leebyron force-pushed the schema-lang-descriptions branch from 32c66a5 to 449992b Compare August 16, 2016 06:21
@leebyron
Copy link
Contributor Author

Note: I need to add this to extendSchema as well.

This adds descriptions to the `buildASTSchema` (string->schema) and `schemaPrinter` (schema->string) via walking the previous full-line contiguous comment block. This is dependent on #463.
@leebyron leebyron force-pushed the schema-lang-descriptions branch from 449992b to 208a11f Compare August 17, 2016 01:52
@leebyron leebyron merged commit 331dc77 into master Aug 24, 2016
@leebyron leebyron deleted the schema-lang-descriptions branch August 24, 2016 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants