Skip to content

remove de-chartered Intl working group#1513

Merged
fhemberger merged 2 commits intonodejs:masterfrom
Trott:rm-intl
Dec 12, 2017
Merged

remove de-chartered Intl working group#1513
fhemberger merged 2 commits intonodejs:masterfrom
Trott:rm-intl

Conversation

@Trott
Copy link
Copy Markdown
Member

@Trott Trott commented Dec 11, 2017

Copy link
Copy Markdown
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@lpinca lpinca left a comment

Choose a reason for hiding this comment

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

LGTM but it should be removed also for other locales.

@Trott
Copy link
Copy Markdown
Member Author

Trott commented Dec 12, 2017

LGTM but it should be removed also for other locales.

Oh, the irony.

Removed it from the only two locales that had it. Kept it as a separate commit in case it ought to be moved to a new PR or something. PTAL.

@Trott
Copy link
Copy Markdown
Member Author

Trott commented Dec 12, 2017

@nodejs/nodejs-ko

@yous
Copy link
Copy Markdown
Contributor

yous commented Dec 12, 2017

LGTM for ko locale.

@fhemberger fhemberger merged commit 576b2fe into nodejs:master Dec 12, 2017
@fhemberger
Copy link
Copy Markdown
Contributor

Thanks! 👍

@srl295
Copy link
Copy Markdown
Member

srl295 commented Dec 12, 2017

@Trott Internationalization: fresh irony served daily.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants