-
Notifications
You must be signed in to change notification settings - Fork 2k
Export Parser
class?
#2741
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Comments
@dotansimha Exported in #2744 note that we plan so major changes to parser implementation as part of merging GraphiQL into import { Parser } from 'graphql/language/parser` Also, can you please share what features is missing in our current parser so we can consider adding it in the future? |
@IvanGoncharov thanks! I wanted to fix a parsing issue related to comments/descriptions. |
@dotansimha Thanks for the detailed explanation 👍 |
Yeah I know, that's the reason we wanted to support it in graphql-tools. To allow developers who still have |
At the moment, the
Parser
class isn't exported at all (https://github.com/graphql/graphql-js/blob/master/src/language/parser.js#L165) which limits the ability to extend or use it as-is.Is there a reason for that?
I would like to extend and add functionality to it, but it seems like it's not possible :(
@IvanGoncharov
The text was updated successfully, but these errors were encountered: