-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Prop filtering #227
Prop filtering #227
Conversation
![core gzip size](http://img.badgesize.io/https://unpkg.com/emotion/dist/DO-NOT-USE.min.js?compression=gzip&label=core%20gzip%20size) | ||
![core size](http://img.badgesize.io/https://unpkg.com/emotion/dist/DO-NOT-USE.min.js?label=core%20size) | ||
![react gzip size](http://img.badgesize.io/https://unpkg.com/emotion/dist/DO-NOT-USE-rect.min.js?compression=gzip&label=react%20gzip%20size) | ||
![react size](http://img.badgesize.io/https://unpkg.com/emotion/dist/DO-NOT-USE-react.min.js?label=react%20size) |
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'm not sure if it's clear enough that the react size includes the core size.
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.
Don't worry about it for the pr. If it looks confusing we can edit the readme later.
Codecov Report
@@ Coverage Diff @@
## master #227 +/- ##
==========================================
+ Coverage 90.27% 90.31% +0.04%
==========================================
Files 20 20
Lines 967 971 +4
Branches 261 263 +2
==========================================
+ Hits 873 877 +4
Misses 76 76
Partials 18 18
Continue to review full report at Codecov.
|
What:
Filter props on string tags.
Why:
#183
How:
Regex generated at build time with
babel-plugin-preval
.Checklist:
This adds about 700b to the react version.