Skip to content

Conversation

@ThePrez
Copy link
Contributor

@ThePrez ThePrez commented May 7, 2019

Opening as a draft, as I figure it makes sense to "land" this when Node 12 is available from IBM (but am willing to take feedback in the meantime).

@ghost ghost marked this pull request as ready for review May 8, 2019 03:18
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

@aymen94
Copy link
Member

aymen94 commented May 9, 2019

LGTM

Maledong added 4 commits May 10, 2019 11:06
zh-CN: translation of "IBM i"
zh-CN: Fix the Italic Words
zh-TW:translation of "IBM i"
en: Fix the Italic Words
@ghost
Copy link

ghost commented May 10, 2019

@nodejs/website:Please have a translation if possible for this, thanks anyway!

osk2 and others added 3 commits May 10, 2019 11:33
@yous
Copy link
Contributor

yous commented May 10, 2019

It seems that * in "*ALLOBJ" is not for italic: https://www.ibm.com/support/knowledgecenter/en/ssw_ibm_i_73/rzarl/rzarlallobjsa.htm

But * can be parsed as a part of markdown syntax, we should use \*ALLOBJ special authority instead of *ALLOBJ special authority.

@yous
Copy link
Contributor

yous commented May 10, 2019

@nodejs/nodejs-ko Updated Korean translation. Please take a look.

@Aissaoui-Ahmed
Copy link
Contributor

Update package-manager.md
@celyes take a look !

@celyes celyes self-requested a review May 11, 2019 10:57
@Aissaoui-Ahmed Aissaoui-Ahmed requested a review from celyes May 11, 2019 11:22
Copy link
Contributor

@celyes celyes left a comment

Choose a reason for hiding this comment

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

Looks great
thank you all @ThePrez @Aissaoui-Ahmed @Maledong @aymen94 @ouadie-lahdioui @mhdawson

@ghost ghost merged commit 4288d92 into nodejs:master May 12, 2019
@ghost
Copy link

ghost commented May 12, 2019

Thank you all!

This pull request was closed.
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.

9 participants