-
Notifications
You must be signed in to change notification settings - Fork 85
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: bump minimum required Node version to 18.14.0
#2420
base: main
Are you sure you want to change the base?
Conversation
Size Change: +1.38 kB (0%) Total Size: 53.4 MB
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2420 +/- ##
=======================================
Coverage 52.47% 52.47%
=======================================
Files 584 584
Lines 22627 22627
Branches 4465 4465
=======================================
Hits 11872 11872
Misses 10721 10721
Partials 34 34 ☔ View full report in Codecov by Sentry. |
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.
thanks!
I believe we will need to add the breaking change
changelog entry again.
fc227c4
to
c58bd36
Compare
Subscribed to pull request
Generated by CodeMention |
18.13.0
18.14.0
This version includes the `File` primitive that might be required for FormData and large file uploads. See: https://nodejs.org/en/blog/release/v18.13.0\#introduce-file See: https://github.com/expo/eas-cli/pull/2414\#discussion_r1634640527
c58bd36
to
ebe6848
Compare
✅ Thank you for adding the changelog entry! |
I bumped the minimum required version to Node 18.14 - seems to be a bug in the This PR will unblock the Undici migration, following the workaround in #2414 (comment) Rebasing that PR onto this one for testing. |
Why
This version includes the
File
primitive that might be required for FormData and large file uploads.See: https://nodejs.org/en/blog/release/v18.13.0#introduce-file
See: #2414 (comment)
How
Bumped all
engines: node: >=18.0.0
toengines: node: >=18.14.0
Test Plan
Only a package change.