Skip to content

multi line strings support#706

Merged
bbakerman merged 10 commits into
graphql-java:masterfrom
bbakerman:685-triple-quoted-strings
Jan 23, 2018
Merged

multi line strings support#706
bbakerman merged 10 commits into
graphql-java:masterfrom
bbakerman:685-triple-quoted-strings

Conversation

@bbakerman
Copy link
Copy Markdown
Member

@bbakerman bbakerman commented Sep 11, 2017

see #685

This implements multi line triple quoted strings and also has SchemaGenerator support and SchemaPrinter support

It doesn't have multi line comments yet
@andimarek
Copy link
Copy Markdown
Member

This will wait a bit more until it the spec is ready (or almost ready)

…-strings

# Conflicts:
#	src/main/java/graphql/schema/idl/SchemaPrinter.java
#	src/test/groovy/graphql/GraphQLTest.groovy
#	src/test/groovy/graphql/parser/ParserTest.groovy
#	src/test/groovy/graphql/schema/idl/SchemaPrinterTest.groovy
@andimarek
Copy link
Copy Markdown
Member

lets reevaluate it again for 6.0

@andimarek andimarek added this to the 6.0 milestone Oct 4, 2017
@andimarek andimarek modified the milestones: 6.0, 7.0 Nov 17, 2017
…-strings

# Conflicts:
#	src/main/java/graphql/language/DirectiveDefinition.java
#	src/main/java/graphql/language/EnumTypeDefinition.java
#	src/main/java/graphql/language/EnumValueDefinition.java
#	src/main/java/graphql/language/FieldDefinition.java
#	src/main/java/graphql/language/InputObjectTypeDefinition.java
#	src/main/java/graphql/language/InputValueDefinition.java
#	src/main/java/graphql/language/InterfaceTypeDefinition.java
#	src/main/java/graphql/language/ObjectTypeDefinition.java
#	src/main/java/graphql/language/ScalarTypeDefinition.java
#	src/main/java/graphql/language/UnionTypeDefinition.java
#	src/main/java/graphql/parser/GraphqlAntlrToLanguage.java
#	src/main/java/graphql/schema/idl/SchemaPrinter.java
#	src/test/groovy/graphql/parser/ParserTest.groovy
@bbakerman bbakerman modified the milestones: 7.0, 8.0 Jan 23, 2018
@bbakerman bbakerman merged commit 092328b into graphql-java:master Jan 23, 2018
exbe pushed a commit to intuit/graphql-java that referenced this pull request Mar 19, 2018
* First commit of multi line strings.

It doesn't have multi line comments yet

* End to end test

* Added description concept to the type definitions

* Schema Generator now supports docstring syntax as a preference

* Refactored out the StringValue parsing into its own class

* SchemaPrinter now has AST description support

* whitespace if defined as ' ' and '\t' in graphql

* Fix ups after merging master
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.

2 participants