-
-
Notifications
You must be signed in to change notification settings - Fork 840
chore(deps): bump wrangler to 4.12.0
#4096
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
Conversation
Bundle size check
Compiler Diagnostics
Reported by octocov |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4096 +/- ##
==========================================
+ Coverage 91.29% 91.33% +0.03%
==========================================
Files 168 168
Lines 10766 10682 -84
Branches 3088 3143 +55
==========================================
- Hits 9829 9756 -73
+ Misses 936 925 -11
Partials 1 1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| }) | ||
| ws.addEventListener('close', async () => { | ||
| closeHandler() | ||
| resolve(undefined) |
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.
It should resolve when in the close event. Otherwise, the test will fail.
4.12.04.12.0
4.12.04.12.0
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! LGTM
Updated
wranglerindevDependenciesto4.12.0from3.58.0for theworkerdruntime test.The author should do the following, if applicable
bun run format:fix && bun run lint:fixto format the code