node icon indicating copy to clipboard operation
node copied to clipboard

Automate timezone updates

Open MylesBorins opened this issue 2 years ago • 21 comments

This happens very infrequently, but would be great to make this automated

Follow up to https://github.com/nodejs/Release/issues/576

MylesBorins avatar Feb 10 '22 15:02 MylesBorins

if you need help with the icu side in CI lmk

srl295 avatar Feb 10 '22 15:02 srl295

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 avatar May 05 '22 11:05 BethGriggs

@BethGriggs probably a good idea.

srl295 avatar May 05 '22 11:05 srl295

^ my offer stands

srl295 avatar May 17 '22 23:05 srl295

Hey @MylesBorins, Can you please brief little more where that issue replicating. Thanks

Ayushi15-Vamne avatar May 23 '22 05:05 Ayushi15-Vamne

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.

MylesBorins avatar May 23 '22 22:05 MylesBorins

I'm interested in solving this issue. Where do I sign up?

jimezesinachi avatar Jul 07 '22 15:07 jimezesinachi

@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?

bnoordhuis avatar Jul 08 '22 12:07 bnoordhuis

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?

srl295 avatar Jul 08 '22 15:07 srl295

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.

jimezesinachi avatar Jul 08 '22 23:07 jimezesinachi

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

srl295 avatar Jul 09 '22 00:07 srl295

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

richardlau avatar Jul 09 '22 00:07 richardlau

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.

srl295 avatar Jul 09 '22 01:07 srl295

I'll get started. Thanks @richardlau & @srl295!

jimezesinachi avatar Jul 09 '22 11:07 jimezesinachi

@rapaktech if you have not started, can I please take on the issue? Kind regards.

98lenvi avatar Jul 21 '22 16:07 98lenvi

Hey @98lenvi! I've been swamped with work the past week, so I haven't been able to start. You're okay to proceed.

jimezesinachi avatar Jul 21 '22 23:07 jimezesinachi

is this issue still open , can anyone tell me how can i help

yashdev9274 avatar Aug 07 '22 10:08 yashdev9274

I thunk its been solved @yashdev9274

jimezesinachi avatar Aug 07 '22 20:08 jimezesinachi

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

yashdev9274 avatar Aug 08 '22 11:08 yashdev9274

if anyone else @bnoordhuis , @MylesBorins , @srl295 , @richardlau , @BethGriggs here can help me please let me know....

yashdev9274 avatar Aug 08 '22 11:08 yashdev9274

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 ?

srl295 avatar Aug 08 '22 12:08 srl295

This is my first time contributing here and I'm not sure what the contributing guideline is. help me

mudassir-jmi avatar Aug 14 '22 05:08 mudassir-jmi

hey, has anyone taken up this issue?

Shub3am avatar Sep 15 '22 20:09 Shub3am

@mudassir-jmi @Shub3am hi. Take a look at https://github.com/nodejs/node/pull/43988 that is where the latest work is

srl295 avatar Sep 15 '22 20:09 srl295