-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Export Parser
class?
#2741
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: