Skip to content
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

weekOfMonth Assert Failure #56

Closed
eric-corumdigital opened this issue May 4, 2021 · 1 comment
Closed

weekOfMonth Assert Failure #56

eric-corumdigital opened this issue May 4, 2021 · 1 comment

Comments

@eric-corumdigital
Copy link

This more or less says all months have 31 days:

return weekOfMonth(year, month - 1, 31);

Which then runs into this assert which says not all months have 31 days:

assert(Number.isInteger(day) && day >= 1 && day <= daysInMonth(year, month), "Argument.Day", "day out of range");

This crashed on May 1st this year.

rogierschouten pushed a commit that referenced this issue May 6, 2021
@rogierschouten
Copy link
Contributor

Hi @eric-corumdigital. thank you for reporting this. It is fixed and published as v5.12.4.

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

No branches or pull requests

2 participants