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

CET is not a variable timezone and should be defined as a UTC offset #564

Open
olivr70 opened this issue Aug 14, 2024 · 1 comment
Open

Comments

@olivr70
Copy link

olivr70 commented Aug 14, 2024

In timezone.ts, CET is defined as mixed timezone, with daylight saving time, like PT (Pacific Time) or some others.

This is not the case. CET is the standard time in Europe, used during the winter. It should just be defined

   CET: 60,

This introduces confusion and unexpected results when using date/time with explicit mention of CET.

This can be verified for example here.

Daylight saving times are extremly complex, and should probably be handled only with IANA Timezones (see (like Europe/Paris). They are supported by the Intl library and by the dayjs module. To get a glimpse of this complexity, one can check How to Read the tz Database Source Files

@wanasit
Copy link
Owner

wanasit commented Jan 25, 2025

Note: This is related to #552 (and #532).

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

No branches or pull requests

2 participants