graphql / graphql-js

A reference implementation of GraphQL for JavaScript
http://graphql.org/graphql-js/
MIT License
20.04k stars 2.02k forks source link

experiment: lazily create DeferredFragments #4150

Closed yaacovCR closed 2 months ago

yaacovCR commented 2 months ago

goal:

avoid creating or passing around the deferMap

methodology:

netlify[bot] commented 2 months ago

Deploy Preview for compassionate-pike-271cb3 ready!

Name Link
Latest commit b4bc441aae5a4662fe3f5a0c4520547466809b8e
Latest deploy log https://app.netlify.com/sites/compassionate-pike-271cb3/deploys/669fc5bc672d1b00081cba51
Deploy Preview https://deploy-preview-4150--compassionate-pike-271cb3.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

github-actions[bot] commented 2 months ago

Hi @yaacovCR, I'm @github-actions bot happy to help you with this PR 👋

Supported commands Please post this commands in separate comments and only one per comment: * `@github-actions run-benchmark` - Run benchmark comparing base and merge commits for this PR * `@github-actions publish-pr-on-npm` - Build package from this PR and publish it on NPM
yaacovCR commented 2 months ago

not sure this is worth it -- but it works!!

yaacovCR commented 2 months ago

closing this for now as best to not modify Path, but we can always revisit