graphql / graphql/graphiql

Make `mergeAST` opt-in only

Open
#836 5 comments 2 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
16.9k
Forks
1.9k
Avg merge
22h 45m
Merged PRs (30d)
70

Description

As per discussions with @IvanGoncharov, the [`utility/mergeAst`](https://github.com/graphql/graphiql/blob/master/src/utility/mergeAst.js) function looks like a good candidate for inclusion in `graphql-js` itself, however it shouldn't be mutating AST directly. Instead, we will refactor it to use the `visit` function to accomplish this, and then open a PR to `graphql-js`. This is just a placeholder for the forthcoming PR

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.