-
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
FSE: Remove obsolete typedefs for @wordpress/icons #42576
Conversation
Caution: This PR affects files in the FSE Plugin on WordPress.com D43838-code has been created so you can easily test it on your sandbox. See this FieldGuide page about developing in the FSE Plugin for more info: PCYsg-ly5-p2 |
This PR does not affect the size of JS and CSS bundles shipped to the user's browser. Generated by performance advisor bot at iscalypsofastyet.com. |
We should remove a bunch of obsolete DT packages as well: DefinitelyTyped/DefinitelyTyped#44673 There are even more than listed in that PR now. |
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.
Seems reasonable… I guess this was included just because there are no files/includes/excludes in the FSE tsconfig?
I'm working on this but can't open a PR right now, GitHub seems to be having problems. https://github.com/Automattic/wp-calypso/tree/remove/obsolete-types |
There we go: #42584 |
Changes proposed in this Pull Request
Follow-up to https://github.com/Automattic/wp-calypso/pull/41703/files#r428944660.
@wordpress/icons
comes with typedefs included these days.Testing instructions
See #41703. (Do a
yarn distclean
first.)