Skip to content

Conversation

@Trott
Copy link
Member

@Trott Trott commented Dec 11, 2017

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

Trott commented Dec 12, 2017

@nodejs/nodejs-ko

@yous
Copy link
Contributor

yous commented Dec 12, 2017

LGTM for ko locale.

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

Thanks! 👍

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