FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

1181 Fix null element in InputValueDefinition#getChildren by tsroka · Pull Request #1182 · graphql-java/graphql-java · GitHub

1181 Fix null element in InputValueDefinition#getChildren - #1182

Merged
andimarek merged 2 commits into
graphql-java:masterfrom
tsroka:1181-fix-null-input-value-def
Aug 23, 2018
Merged

1181 Fix null element in InputValueDefinition#getChildren#1182
andimarek merged 2 commits into
graphql-java:masterfrom
tsroka:1181-fix-null-input-value-def

Conversation

tsroka commented Aug 21, 2018

Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown
Member

@tsroka how did you come across this?

Copy link
Copy Markdown
Member

@tsroka could u use the graphql-java formatter (https://github.com/graphql-java/graphql-java/blob/master/graphql-java-code-style.xml) to avoid not needed changes? thanks

tsroka commented Aug 21, 2018

Copy link
Copy Markdown
Author

@andimarek while working on nadel, i needed to traverse AST using NodeTraverser and got NPE. Will fix formatting,

Copy link
Copy Markdown
Member

this should also go into stable 9.x

andimarek added this to the 10.0 milestone Aug 23, 2018
andimarek merged commit 141b8e4 into graphql-java:master Aug 23, 2018
bbakerman added the needs to be backported a bugfix that still needs to be backported label Aug 30, 2018
bbakerman added a commit to bbakerman/graphql-java that referenced this pull request Aug 30, 2018
bbakerman mentioned this pull request Aug 30, 2018
bbakerman added a commit that referenced this pull request Aug 30, 2018
andimarek removed the needs to be backported a bugfix that still needs to be backported label Sep 3, 2018
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

4 participants


Back | FazBrowse Home | New Git URL