-
Notifications
You must be signed in to change notification settings - Fork 30.6k
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
Automate timezone updates #43134
Comments
if you need help with the icu side in CI lmk |
Curious if we should consider moving this to |
@BethGriggs probably a |
^ my offer stands |
Hey @MylesBorins, |
we embed icu in Node.js which has timezone information embedded. This timezone data can end up being updated separately from the icu dependency being updated, this is especially important for LTS versions of Node.js which may not get ICU updates regularly. It is important that the timezone data is up to date or it can cause programming erorrs or issues with services. This issue referenced in the original post does a good job of explaining how things can go bad. |
I'm interested in solving this issue. Where do I sign up? |
@rapaktech Right here. :-) Node's tzdata comes from ICU. ICU's binary data file is stored in deps/icu-small/source/data/in. Somehow that file should be updated to include newer tzdata - maybe by updating ICU's data/zone/* files from icu4c-71_1-data.zip and building a new .dat file? |
ICU has a whole guide on this topic,
https://unicode-org.github.io/icu/userguide/datetime/timezone/#updating-the-time-zone-data
And I thought node has a guide as well?
|
Thanks @bnoordhuis & @srl295. Gone through the resources you shared, and I think I understand what to do. This is my first time contributing here and I'm not sure what the contributing guideline is. Would appreciate any pointers. Thanks. |
Great
Start here https://github.com/nodejs/node/blob/main/CONTRIBUTING.md |
Yes, https://github.com/nodejs/node/blob/main/doc/contributing/maintaining-icu.md#time-zone-data |
I’d start with manually following the nodejs update instructions
…Sent from my iPad
On Jul 8, 2022, at 7:14 PM, Richard Lau ***@***.***> wrote:
ICU has a whole guide on this topic, https://unicode-org.github.io/icu/userguide/datetime/timezone/#updating-the-time-zone-data And I thought node has a guide as well?
Yes, https://github.com/nodejs/node/blob/main/doc/contributing/maintaining-icu.md#time-zone-data
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.
|
I'll get started. Thanks @richardlau & @srl295! |
@rapaktech if you have not started, can I please take on the issue? Kind regards. |
Hey @98lenvi! I've been swamped with work the past week, so I haven't been able to start. You're okay to proceed. |
is this issue still open , can anyone tell me how can i help |
I thunk its been solved @yashdev9274 |
thank you @rapaktech for mentioning , are you working on any project currently which i can help to contribute |
if anyone else @bnoordhuis , @MylesBorins , @srl295 , @richardlau , @BethGriggs here can help me please let me know.... |
#43988 is still open. Can you comment on that ? |
This is my first time contributing here and I'm not sure what the contributing guideline is. help me |
hey, has anyone taken up this issue? |
@mudassir-jmi @Shub3am hi. Take a look at #43988 that is where the latest work is |
Fixes: nodejs#43134 PR-URL: nodejs#43988 Reviewed-By: Steven R Loomis <srloomis@us.ibm.com>
Fixes: nodejs/node#43134 PR-URL: nodejs/node#43988 Reviewed-By: Steven R Loomis <srloomis@us.ibm.com>
Fixes: nodejs/node#43134 PR-URL: nodejs/node#43988 Reviewed-By: Steven R Loomis <srloomis@us.ibm.com>
This happens very infrequently, but would be great to make this automated
Follow up to nodejs/Release#576
The text was updated successfully, but these errors were encountered: