-
-
Notifications
You must be signed in to change notification settings - Fork 289
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
chore: downgrade cross-fetch to v3 #5806
Conversation
Performance Report✔️ no performance regression detected Full benchmark results
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm 🚀
Downgrading cross-fetch for now is good but this does not resolve #5765. Still need to update node-fetch to a version that has the fix applied. Ideally, we want them to backport it to v2. |
🎉 This PR is included in v1.10.0 🎉 |
Motivation
cross-fetch v4 causes browser issues
and right now seems to be relatively low usage compared to v3, it is better to delay the update for a while.
Description