-
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
Gutenboarding: Performance Improvements #40369
Comments
PR with some improvements for |
This issue is stale because it has been 180 days with no activity. You can keep the issue open by adding a comment. If you do, please provide additional context and explain why you’d like it to remain open. You can also close the issue yourself — if you do, please add a brief explanation and apply one of relevant issue close labels. |
@sgomes, checking in for an update on this issue. Is is closable (as complete or otherwise)? For context, several of us are working on a cross-functional group triage effort in Dotcom to help us better manage our issue backlog. Thanks! |
Hey @inaikem! 👋 Sure, happy to help! I'm sure that any findings are woefully out of date or entirely inapplicable at this point. No need to keep this around, so I'll close as not planned 👍 |
For context, Gutenboarding was removed a while ago, making this issue no longer relevant: #74475 |
@sgomes looked into Gutenboarding's performance and reported a few items that could need improvement (p1584972373304500-slack-gutenboarding):
showdown
is pretty big, at 23KB.react-dates
is an absurd 37.5KB.moment-timezone
is also big. (These need upstream fixing in Gutenberg).wpcom
:getCart
andsetCart
. Should replace by directly usingwpcom-proxy-request
.eslint
rule against importingwpcom
(maybe Calypso-wide; I guess we can consider this legacy and recommend against it) /cc @Automattic/team-calypso (ESLint: Restrictimport
s andrequire
s fromlib/wp
#40371)lib/analytics
. People have started moving stuff into https://github.com/Automattic/wp-calypso/tree/master/packages/calypso-analytics; @sgomes will look into this.The text was updated successfully, but these errors were encountered: