-
-
Notifications
You must be signed in to change notification settings - Fork 825
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
[NFC] test fix for leap year #29515
[NFC] test fix for leap year #29515
Conversation
🤖 Thank you for contributing to CiviCRM! ❤️ We will need to test and review this PR. 👷 Introduction for new contributors...
Quick links for reviewers...
|
Oh yes, probably every day between feb 26 and feb 28. Ok let me get my time machine and rejig. |
The feb 29 I couldn't do in my head. But it can be confirmed by seeing that:
and
returns 4 |
Ah I should put this against 5.71, otherwise tests on backports will fail for a few days. |
Happy 29th of Feb |
I still can't get over how you're living in the future. I think I'm going to propose to the [insert whatever org controls time] that individual regions be allowed to regulate if they want leap years, since our time systems are too simple with just timezones and DST. |
shame it doesn't allow me to tell you who will win lotto |
Overview
test fails on Feb 26 in leap years.