-
Notifications
You must be signed in to change notification settings - Fork 7
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
Upgrade zen-fs deps #41
Conversation
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.
unit test failures look similar to #40.
package.json
Outdated
@@ -60,7 +60,7 @@ | |||
"gzip": "3.5 KB" | |||
}, | |||
"dist/play-pen.js": { | |||
"none": "26580 KB", | |||
"none": "26817 KB", |
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 surprised the uncompressed size increased so much and the compressed didn't need to change at all.
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.
The size test doesn't fail locally, so I based the size on the number reported by the CI runner π€·
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.
thank you!
Closes #
πΈ TL;DR
Migrates the unsupported dependency, follow up for #40
π Details
Design Doc
Jira
π§ͺ Testing Steps / Validation
β Checks