search

graphql-go / graphql

An implementation of GraphQL for Go / Golang
MIT License
9.82k stars 836 forks source link

Expose ParseValue #656

Closed victor-perov closed 1 year ago

victor-perov commented 1 year ago

This PR marks ParseValue as a public method that allows it to be called directly. It is handy to get AST value as part of pre-processing / validation process.

coveralls commented 1 year ago

Coverage Status

Coverage remained the same at 92.051% when pulling 2ce8c8b2bad853b09de23d8153d5c7f8436871b1 on victor-perov:victor/expose-ParseValue into 09272f35006712d597c0446258874ce7ff8919fb on graphql-go:master.

victor-perov commented 1 year ago

Hi @chris-ramon @sogko! Would you mind taking a look at this PR, please? I would appreciate if we can get it merged.

Kind regards, Victor

victor-perov commented 1 year ago

Is there any chance this change can be reviewed?

chris-ramon commented 1 year ago

Thanks a lot @victor-perov

victor-perov commented 1 year ago

Thanks for merging, @chris-ramon!