Skip to content
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

[pull] main from apollographql:main #157

Merged
merged 9 commits into from
May 24, 2022
Merged

[pull] main from apollographql:main #157

merged 9 commits into from
May 24, 2022

Conversation

pull[bot]
Copy link

@pull pull bot commented May 22, 2022

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

@pull pull bot added the ⤵️ pull label May 22, 2022
charle692 and others added 8 commits May 23, 2022 11:16
…mpleted and onError could not be changed while a mutation is being executed.

Delete .dccache
Issue #9735 appears to be due to a bug I introduced in my commit
756ab87 in PR #8734.

Premature optimization has a bad reputation already, but you can add
this PR to the mountain of evidence in its disfavor.
If you revert the previous commit and run `npm test`, you'll see all the
tests this dynamic Object.isFrozen check has been silently protecting
from failing, but only (and this is the important part) in development.

Since we only actually freeze objects with Object.freeze in development,
this Object.isFrozen check does not help in production, so an object
that would have been frozen in development gets reused as a mutable
copy, potentially acquiring properties it should not acquire (a bug
fixed by the previous commit, first reported in issue #9735).
…d-only-mismerge

Restrict `DeepMerger` mutable object reuse to fix subtle production-only bug
…with `variables` passed to execution function (#9758)
@pull pull bot merged commit b86c363 into NOUIY:main May 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants