Skip to content

Upgrade devtools dependency update-notifier to 5.x#31655

Merged
hoxyq merged 1 commit intofacebook:mainfrom
stianjensen:upgrade-update-notifier
Nov 7, 2025
Merged

Upgrade devtools dependency update-notifier to 5.x#31655
hoxyq merged 1 commit intofacebook:mainfrom
stianjensen:upgrade-update-notifier

Conversation

@stianjensen
Copy link
Contributor

Summary

This PR upgrades the dependency on update-notifier, used in react-devtools, to 5.x
This is the latest non-ESM version, so upgrading to it should be unproblematic (while updating to 6.x and beyond will have to wait).

Upgrading means we avoid installing a lot of outdated dependencies (as can be seen from the diff in yarn.lock), and resolves part of #28058

Changelog:
https://github.com/yeoman/update-notifier/releases

The most relevant breaking change seems to be that the minimum support node version is increased from v6 to v10, but I couldn't find what is currently React's official node version support.

How did you test this change?

I ran the test-suite locally (yarn test in root folder), but I'm not sure if that one actually covers devtools?

I also built and tested this version of devtools with some internal company projects (both react and react-native based) – following guidelines from #28058 (comment).

@vercel
Copy link

vercel bot commented Dec 2, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
react-compiler-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 2, 2024 10:16pm

@rickhanlonii
Copy link
Member

Thanks! For security reasons, someone on the team should update the lockfile manually before merging this. The changes are too big for me to review manually.

cc @hoxyq @eps1lon

This is the latest non-ESM version, so upgrading to it should be unproblematic
(while updating to 6.x and beyond will have to wait).

Upgrading means we avoid installing a lot of outdated dependencies, and
resolves part of facebook#28058

Changelog:
https://github.com/yeoman/update-notifier/releases

The most relevant breaking change seems to be that the minimum support node
version is increased from v6 to v10, but I couldn't find what is currently
React's official node version support.
@stianjensen stianjensen force-pushed the upgrade-update-notifier branch from a74c139 to c9ea15d Compare November 5, 2025 12:26
@react-sizebot
Copy link

Comparing: dd048c3...c9ea15d

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.js = 6.68 kB 6.68 kB = 1.83 kB 1.83 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 607.89 kB 607.89 kB = 107.61 kB 107.61 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.69 kB 6.69 kB = 1.83 kB 1.83 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 665.84 kB 665.84 kB = 117.34 kB 117.34 kB
facebook-www/ReactDOM-prod.classic.js = 691.67 kB 691.67 kB = 121.70 kB 121.70 kB
facebook-www/ReactDOM-prod.modern.js = 682.09 kB 682.09 kB = 120.08 kB 120.08 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against c9ea15d

Copy link
Contributor

@hoxyq hoxyq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@hoxyq hoxyq merged commit a10ff9c into facebook:main Nov 7, 2025
240 checks passed
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.

5 participants