Skip to content

[12.x] tools: update tzdata to 2019c #30478

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

[12.x] tools: update tzdata to 2019c #30478

wants to merge 1 commit into from

Conversation

@MylesBorins
Copy link
Contributor

@MylesBorins MylesBorins commented Nov 13, 2019

Refs: #30211
Refs: #30356

@MylesBorins MylesBorins requested review from srl295 and targos November 13, 2019 21:48
@MylesBorins MylesBorins changed the title tools: update tzdata to 2019c [12.x] tools: update tzdata to 2019c Nov 13, 2019
srl295
srl295 approved these changes Nov 13, 2019
View reviewed changes
Copy link
Member

@srl295 srl295 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM…

@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Nov 13, 2019

CI: https://ci.nodejs.org/job/node-test-pull-request/26573/

@MylesBorins MylesBorins mentioned this pull request Nov 13, 2019
Merged
@MylesBorins
Copy link
Contributor Author

MylesBorins commented Nov 13, 2019

Could we please fast-track this

@MylesBorins MylesBorins added the fast-track PRs that do not need to wait for 48 hours to land. label Nov 13, 2019
@targos
Copy link
Member

targos commented Nov 14, 2019

Landed in 2810f1a

@targos targos closed this Nov 14, 2019
targos pushed a commit that referenced this pull request Nov 14, 2019
@MylesBorins @targos
tools: update tzdata to 2019c
2810f1a 
Refs: #30211
Refs: #30356

PR-URL: #30478
Reviewed-By: Steven R Loomis <srloomis@us.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@srl295 srl295 srl295 approved these changes

@targos targos targos approved these changes

@cjihrig cjihrig cjihrig approved these changes

Assignees

No one assigned

Labels

fast-track PRs that do not need to wait for 48 hours to land.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@MylesBorins @nodejs-github-bot @targos @srl295 @cjihrig