Repository navigation
Fix main: replace jsg::Number with f64 in jsg::Dict tests - #8071
Conversation
|
✅ Approved — This is a mechanical swap from The decision is advisory: this comment never approves, blocks or requests changes on the pull request. ✅ All checks passed📋 Code Quality: ✅ LGTMThe latest commit adds The rest of the diff is a mechanical replacement of 📝 Documentation: ✅ LGTMThe Reviewed commit: |
|
Since last review: 0 resolved, 0 still open, 0 new. Not re-run: tests, api-compat, docs, compat-flags, design-simplicity, jsg-gc, memory-safety, rust-first (no author changes in their files since the last review) Reviewed commit: 7b1830a4 · github run |
|
The generated output of |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8071 +/- ##
==========================================
- Coverage 69.49% 69.42% -0.07%
==========================================
Files 695 696 +1
Lines 181564 181742 +178
Branches 29204 29211 +7
==========================================
+ Hits 126170 126180 +10
- Misses 38328 38496 +168
Partials 17066 17066 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Fixing main because of conflict between #8054 and #7998.