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

fix: prevent context fetcherOptions being overwritten by mutate #227

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

dan-cooke
Copy link

Fixes #226

@dan-cooke
Copy link
Author

dan-cooke commented Mar 11, 2024

I am having difficulties updating the snapshots locally, I am getting strange jest runtime errors


Error: You need to run with a version of node that supports ES Modules in the VM API. See https://jestjs.io/docs/ecmascript-modules

If anyone else is able to pull this branch, update the snapshots and merge this PR in that would be great

@dan-cooke
Copy link
Author

I'm also unsure about the dependency on lodash/merge , can probably copy it into the repo actually

@Cellule
Copy link

Cellule commented Mar 15, 2024

To update snapshots you have 2 options, wait on #234
Or simpler, temporarily downgrade pretter to v2, update the snapshot then revert prettier back to v3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Headers from context are overwritten from mutate headers
2 participants