Skip to content

fix: correct isJSONSerializable to handle null without throwing#572

Open
EduardF1 wants to merge 2 commits into
unjs:mainfrom
EduardF1:fix/isJSONSerializable-null
Open

fix: correct isJSONSerializable to handle null without throwing#572
EduardF1 wants to merge 2 commits into
unjs:mainfrom
EduardF1:fix/isJSONSerializable-null

test: cover null JSON serialization

a6cf69a
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jun 30, 2026 in 1s

85.55% (+28.68%) compared to 27996d3

View this Pull Request on Codecov

85.55% (+28.68%) compared to 27996d3

Details

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.55%. Comparing base (27996d3) to head (a6cf69a).
⚠️ Report is 103 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #572       +/-   ##
===========================================
+ Coverage   56.86%   85.55%   +28.68%     
===========================================
  Files          16        5       -11     
  Lines         728      263      -465     
  Branches      113      133       +20     
===========================================
- Hits          414      225      -189     
+ Misses        303       33      -270     
+ Partials       11        5        -6     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.