Skip to content

Conversation

@catamorphism
Copy link
Contributor

The polyfill always calls ISODateSurpasses with weeks and days equal to 0, and the arguments are only used when they're not equal to 0, so remove them altogether. (In the spec, ISODateSurpasses is called with non-zero weeks and days, but the polyfill does an optimization that means it's not called with non-zero weeks or days.)

The polyfill always calls ISODateSurpasses with weeks and days
equal to 0, and the arguments are only used when they're not equal to
0, so remove them altogether. (In the spec, ISODateSurpasses is called
with non-zero weeks and days, but the polyfill does an optimization that
means it's not called with non-zero weeks or days.)
@codecov
Copy link

codecov bot commented Dec 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.83%. Comparing base (03f9057) to head (de98c17).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3214      +/-   ##
==========================================
+ Coverage   96.75%   96.83%   +0.07%     
==========================================
  Files          22       22              
  Lines       10398    10390       -8     
  Branches     1859     1858       -1     
==========================================
  Hits        10061    10061              
+ Misses        289      281       -8     
  Partials       48       48              

☔ View full report in Codecov by Sentry.
📢 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant