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

rename for deprecated timezones #634

Closed
wants to merge 1 commit into from

Conversation

JonBendtsen
Copy link

@phil-davis
Copy link
Contributor

@JonBendtsen one unit test fails, something related to timezones.
Please have a look. Maybe the test needs to change, or maybe the updated timezone data needs a small tweak?

@JonBendtsen
Copy link
Author

@JonBendtsen one unit test fails, something related to timezones. Please have a look. Maybe the test needs to change, or maybe the updated timezone data needs a small tweak?

@phil-davis
I have now looked at the reports from PHPUnit for all of the different PHP versions and I can not in that output see anything that indicates precisely where it goes wrong, so I can not even see where I should make a small tweak.

'Brazil/West',
'Canada/Atlantic',
'Canada/Central',
'Canada/Eastern',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The failing unit test happens to use Canada/Eastern, which is removed here.
https://www.php.net/manual/en/timezones.others.php
"Warning: Please do not use any of the timezones listed here (besides UTC), they only exist for backward compatible reasons, and may expose erroneous behavior. Furthermore, these timezones may be removed from the IANA timezone database at any time."

And Canada/Eastern is listed there.

The unit test can be fixed by using a "proper" timezone in the test. But we need to think and properly understand what happens in general if we remove these timezones from the data here.

@JonBendtsen JonBendtsen closed this by deleting the head repository Apr 19, 2024
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.

None yet

2 participants