node
node copied to clipboard
Automate timezone updates
This happens very infrequently, but would be great to make this automated
Follow up to https://github.com/nodejs/Release/issues/576
if you need help with the icu side in CI lmk
Curious if we should consider moving this to nodejs/node
(and potentially tagging with good first issue
?). The automation would be on that repository, and this issue would probably get more visibility there.
@BethGriggs probably a good idea
.
^ my offer stands
Hey @MylesBorins, Can you please brief little more where that issue replicating. Thanks
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.
Thanks @bnoordhuis & @srl295. Gone through the resources you shared, and I think I understand what to do.
Great
This is my first time contributing here and I'm not sure what the contributing guideline is. Would appreciate any pointers. Thanks.
Start here https://github.com/nodejs/node/blob/main/CONTRIBUTING.md
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
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 My tech stack is MERN stack and ML , please let me know . I want to start my open source journey but I have no one to guide me , if you can guide me than I will be able to improve my developing skills with open source and can get a job
if anyone else @bnoordhuis , @MylesBorins , @srl295 , @richardlau , @BethGriggs here can help me please let me know....
if anyone else @bnoordhuis , @MylesBorins , @srl295 , @richardlau , @BethGriggs here can help me please let me know....
https://github.com/nodejs/node/pull/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 https://github.com/nodejs/node/pull/43988 that is where the latest work is