Comments / Descriptions about graphql HOT 3 CLOSED

Hi @alpox!, thumbs up for graphql-go-gen pretty useful project 👍 & really looking forward to try it out.

In regards graphql/language/parser/parser.go ignoring comments, I think its the expected behavior, the next following points confirms it, which are based on the graphql-js reference implementation:

• There is not a comment TokenKind listed on the possible tokens.

• readToken from graphql-js/src/language/lexer.js ignores comments:

  This skips over whitespace and comments until it finds the next lexable token ...

  Source: https://github.com/graphql/graphql-js/blob/master/src/language/lexer.js#L139

• There is a test for graphql-js/src/language/lexer.js which effectively includes comments on the query and they are ignored by the lexer & parser.

One workaround would be to copy lexer & parser and include those to graphql-go-gen, then extend their functionality to not ignore comments, perhaps starting from func positionAfterWhitespace(body string, startPosition int) int { ... }, similar to github.com/ForbesLindesay/graphql-schema-gen strategy.

Keep it up the great work!

from graphql.

Thanks @chris-ramon for your response. I will try to do like you suggested. It could take me a while to finish since i'm quite busy these days and i'm not familiar with compilers 😃 I'll try my best!

from graphql.

Alright @alpox, take it easy, keep up the great work and feel free to re-open the issue as needed.

from graphql.