-
Notifications
You must be signed in to change notification settings - Fork 2k
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(deps): update wordpress monorepo #57217
Conversation
Link to Calypso live: https://calypso.live?image=registry.a8c.com/calypso/app:build-19898 |
This PR modifies the release build for editing-toolkit To test your changes on WordPress.com, run To deploy your changes after merging, see the documentation: PCYsg-mMA-p2 |
Here is how your PR affects size of JS and CSS bundles shipped to the user's browser: App Entrypoints (~66393 bytes added 📈 [gzipped])
Common code that is always downloaded and parsed every time the app is loaded, no matter which route is used. Sections (~6673 bytes removed 📉 [gzipped])
Sections contain code specific for a given set of routes. Is downloaded and parsed only when a particular route is navigated to. Async-loaded Components (~173624 bytes added 📈 [gzipped])
React components that are loaded lazily, when a certain part of UI is displayed for the first time. Legend What is parsed and gzip size?Parsed Size: Uncompressed size of the JS and CSS files. This much code needs to be parsed and stored in memory. Generated by performance advisor bot at iscalypsofastyet.com. |
53d0710
to
784618d
Compare
697a975
to
ad17674
Compare
8f904f4
to
87aa612
Compare
6f6baf8
to
c73db9f
Compare
Current status: I fixed all problems (afaik) related to missing peer dependencies, duplicated packages, compilation errors, etc. I have not investigated the E2E errors or verified the functionality of the site. |
c73db9f
to
8981986
Compare
There was a conflict with the react-native update, so I rebased. There is one remaining peer dependency issue now, also related to react native. I looked into it again, and we are so close to being able to remove react native everywhere. None of the updated wordpress packages seem to require react-native any more, but some of our transitive dependencies still rely on outdated versions of edit: this DT PR should fix the transitive dependency: DefinitelyTyped/DefinitelyTyped#57085 |
8981986
to
b9cb9a2
Compare
There are two pending issues: E2E errors They are real and easy to reproduce. Just go to Bundle size The increase in |
I investigated the bundle size increase for the The The
|
Thanks for the investigation. Maybe we should do #57825 at the same time to avoid bloating isolated-block-editor 🤔 |
It's async loaded and it's a rarely-used experimental feature: wp-calypso/client/blocks/comments/form.jsx Lines 179 to 185 in f2e6f88
So it theoretically doesn't have a big impact on production. Either way, it'd be easy to do that upgrade at the same time |
b9cb9a2
to
cc7be30
Compare
@scinos Sorry I wasn't able to pitch in earlier, but it looks like @jsnajdr already performed an investigation in my stead. Thank you @jsnajdr! I need to add that the approach that There is a lot potential for these packages to have a much smaller impact in their consumers, if they're re-organised to have less interdependent code, or perhaps split out further into smaller packages. |
FWIW, I listed all outdated
Once we update to a version of |
Definitely. Though, one challenge is keeping all of the other consumers up to date as well. For example, I believe we will still be on one version newer WordPress packages compared to newspack by the time we can get that package updated. |
5ab67be
to
1e1c3d6
Compare
1e1c3d6
to
a075f07
Compare
eeaeb77
to
7e56eef
Compare
The current error is that e2e tests can't change the language in the account page. It works for me in calypso.localhost, so unsure what the exact problem is. I am seeing this warning on the account page:
The screenshot in TeamCity just shows a blank screen. I guess that could mean that the page reload isn't finishing? |
The E2E error is real, I can reproduce it in calypso.live More details in #57217 (comment) |
The crash in |
This one, in the Me > Account Settings screen, exists even before this PR. It's caused by the |
This warning is being fixed in #58236 by migrating away from |
0e3a604
to
a8de17d
Compare
Since gutenboarding (IIRC) is strictly typed, it needed a type definition for I also took the opportunity to resolve a TODO comment I had left during a previous wp monorepo upgrade, which said to move a .scss import back into the TS file from the SCSS file. |
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.
I think this is good to go! The ETK changes are very minimal. Smoke testing gutenboarding/calypso, everything seems okay.
* fix(deps): update wordpress monorepo * Fixes yarn warnings about peer dependencies * Fix tests that require jsdom environment * Use `sass:math` * Update @automattic/isolated-block-editor * Dedupe packages * Mount ShortcutProvider to avoid useShortcut crash * Resolve TODO related to wordpress styles * Declare types for wp/keyboard-shortcuts * Remove comments Co-authored-by: Renovate Bot <[email protected]> Co-authored-by: Sergio Cinos <[email protected]> Co-authored-by: Noah Allen <[email protected]> Co-authored-by: Jarda Snajdr <[email protected]>
This PR contains the following updates:
^3.2.2
->^3.2.3
^5.2.2
->^5.2.5
^4.0.0
->^4.0.3
^7.0.2
->^8.0.0
^11.1.0
->^11.1.3
^17.0.0
->^19.0.1
^5.0.2
->^5.0.5
^6.1.0
->^6.1.3
^2.2.3
->^2.2.6
^4.2.1
->^4.2.2
^3.2.1
->^3.2.2
^3.2.1
->^3.2.2
^5.0.2
->^5.0.5
^3.0.2
->^3.0.5
^11.0.2
->^12.0.2
^4.0.1
->^4.0.3
^4.1.1
->^4.1.3
^2.2.1
->^2.2.2
^9.1.2
->^9.2.0
^3.0.2
->^3.0.5
^3.2.0
->^3.2.1
^3.2.1
->^3.2.2
^4.2.2
->^4.2.3
^5.0.2
->^6.1.0
^7.1.1
->^7.1.2
^3.2.2
->^3.2.3
^5.0.2
->^5.0.5
^4.0.2
->^4.0.5
^3.0.1
->^3.0.3
^3.0.1
->^3.0.3
^5.0.2
->^5.0.5
^18.0.1
->^19.1.0
^3.0.2
->^3.0.5
^3.2.2
->^3.3.0
^4.0.2
->^4.0.5
^2.2.1
->^2.2.2
Release Notes
WordPress/gutenberg
v3.2.3
Compare Source
Configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.
This PR has been generated by WhiteSource Renovate. View repository job log here.