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

Automate timezone updates #43134

Closed
MylesBorins opened this issue Feb 10, 2022 · 24 comments · Fixed by #43988
Closed

Automate timezone updates #43134

MylesBorins opened this issue Feb 10, 2022 · 24 comments · Fixed by #43988
Labels
good first issue Issues that are suitable for first-time contributors. help wanted Issues that need assistance from volunteers or PRs that need help to proceed.

Comments

@MylesBorins
Copy link
Member

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

Follow up to nodejs/Release#576

@srl295
Copy link
Member

srl295 commented Feb 10, 2022

if you need help with the icu side in CI lmk

@BethGriggs
Copy link
Member

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 BethGriggs added the release-agenda Issues and PRs to discuss during the meetings of the Release team. label May 5, 2022
@srl295
Copy link
Member

srl295 commented May 5, 2022

@BethGriggs probably a good idea.

@BethGriggs BethGriggs removed the release-agenda Issues and PRs to discuss during the meetings of the Release team. label May 17, 2022
@BethGriggs BethGriggs transferred this issue from nodejs/Release May 17, 2022
@BethGriggs BethGriggs added good first issue Issues that are suitable for first-time contributors. help wanted Issues that need assistance from volunteers or PRs that need help to proceed. labels May 17, 2022
@srl295
Copy link
Member

srl295 commented May 17, 2022

^ my offer stands

@Ayushi15-Vamne
Copy link

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

@MylesBorins
Copy link
Member Author

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.

@jimezesinachi
Copy link

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

@bnoordhuis
Copy link
Member

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

@srl295
Copy link
Member

srl295 commented Jul 8, 2022 via email

@jimezesinachi
Copy link

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.

@srl295
Copy link
Member

srl295 commented Jul 9, 2022

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

@richardlau
Copy link
Member

@srl295
Copy link
Member

srl295 commented Jul 9, 2022 via email

@jimezesinachi
Copy link

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

@98lenvi
Copy link
Contributor

98lenvi commented Jul 21, 2022

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

@jimezesinachi
Copy link

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

98lenvi added a commit to 98lenvi/node that referenced this issue Jul 25, 2022
98lenvi added a commit to 98lenvi/node that referenced this issue Jul 25, 2022
98lenvi added a commit to 98lenvi/node that referenced this issue Jul 25, 2022
98lenvi added a commit to 98lenvi/node that referenced this issue Jul 26, 2022
98lenvi added a commit to 98lenvi/node that referenced this issue Jul 26, 2022
98lenvi added a commit to 98lenvi/node that referenced this issue Jul 26, 2022
@yashdev9274
Copy link

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

@jimezesinachi
Copy link

I thunk its been solved @yashdev9274

@yashdev9274
Copy link

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
Copy link

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

@srl295
Copy link
Member

srl295 commented Aug 8, 2022

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 ?

98lenvi added a commit to 98lenvi/node that referenced this issue Aug 9, 2022
98lenvi added a commit to 98lenvi/node that referenced this issue Aug 9, 2022
98lenvi added a commit to 98lenvi/node that referenced this issue Aug 9, 2022
@mudassir-jmi
Copy link

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

@Shub3am
Copy link

Shub3am commented Sep 15, 2022

hey, has anyone taken up this issue?

@srl295
Copy link
Member

srl295 commented Sep 15, 2022

@mudassir-jmi @Shub3am hi. Take a look at #43988 that is where the latest work is

aduh95 pushed a commit that referenced this issue Sep 15, 2022
Fixes: #43134
PR-URL: #43988
Reviewed-By: Steven R Loomis <srloomis@us.ibm.com>
Fyko pushed a commit to Fyko/node that referenced this issue Sep 15, 2022
Fixes: nodejs#43134
PR-URL: nodejs#43988
Reviewed-By: Steven R Loomis <srloomis@us.ibm.com>
RafaelGSS pushed a commit that referenced this issue Sep 26, 2022
Fixes: #43134
PR-URL: #43988
Reviewed-By: Steven R Loomis <srloomis@us.ibm.com>
RafaelGSS pushed a commit that referenced this issue Sep 26, 2022
Fixes: #43134
PR-URL: #43988
Reviewed-By: Steven R Loomis <srloomis@us.ibm.com>
RafaelGSS pushed a commit that referenced this issue Sep 26, 2022
Fixes: #43134
PR-URL: #43988
Reviewed-By: Steven R Loomis <srloomis@us.ibm.com>
juanarbol pushed a commit that referenced this issue Oct 4, 2022
Fixes: #43134
PR-URL: #43988
Reviewed-By: Steven R Loomis <srloomis@us.ibm.com>
juanarbol pushed a commit that referenced this issue Oct 4, 2022
Fixes: #43134
PR-URL: #43988
Reviewed-By: Steven R Loomis <srloomis@us.ibm.com>
juanarbol pushed a commit that referenced this issue Oct 4, 2022
Fixes: #43134
PR-URL: #43988
Reviewed-By: Steven R Loomis <srloomis@us.ibm.com>
juanarbol pushed a commit that referenced this issue Oct 7, 2022
Fixes: #43134
PR-URL: #43988
Reviewed-By: Steven R Loomis <srloomis@us.ibm.com>
juanarbol pushed a commit that referenced this issue Oct 10, 2022
Fixes: #43134
PR-URL: #43988
Reviewed-By: Steven R Loomis <srloomis@us.ibm.com>
juanarbol pushed a commit that referenced this issue Oct 11, 2022
Fixes: #43134
PR-URL: #43988
Reviewed-By: Steven R Loomis <srloomis@us.ibm.com>
guangwong pushed a commit to noslate-project/node that referenced this issue Jan 3, 2023
Fixes: nodejs/node#43134
PR-URL: nodejs/node#43988
Reviewed-By: Steven R Loomis <srloomis@us.ibm.com>
guangwong pushed a commit to noslate-project/node that referenced this issue Jan 3, 2023
Fixes: nodejs/node#43134
PR-URL: nodejs/node#43988
Reviewed-By: Steven R Loomis <srloomis@us.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Issues that are suitable for first-time contributors. help wanted Issues that need assistance from volunteers or PRs that need help to proceed.
Projects
None yet
Development

Successfully merging a pull request may close this issue.