-
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
introduces mapSchemaConfig utility function #4297
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for compassionate-pike-271cb3 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Hi @yaacovCR, I'm @github-actions bot happy to help you with this PR 👋 Supported commandsPlease post this commands in separate comments and only one per comment:
|
e747264
to
23e7952
Compare
note: diff is easier to scan when hiding whitespace |
extracting this from lexicographicallySortSchema and extendSchemaImpl motivation: 1. to use to further enhance extendSchema/buildASTSchema/buildSchema to take resolvers, etc, and 2. to expose a generic safe mapSchemaConfig
Just poking around a bit with deopt-explorer, which, as usual, is beyond me, but the summary for main is: and the summary for map is: For clarity, within the map version, we rename For clarity, I also renamed So the thing that leaps out at me is that the number of deoptimizations in |
motivation:
TO DO: improve test coverage