Skip to content

Change Chinese Translation to a newer version - #352

Merged
hshoff merged 1 commit into
airbnb:masterfrom
sivan:master
Jun 17, 2015
Merged

Change Chinese Translation to a newer version#352
hshoff merged 1 commit into
airbnb:masterfrom
sivan:master

Conversation

@sivan

@sivan sivan commented May 25, 2015

Copy link
Copy Markdown
Contributor

The current Chinese version translated by adamlu is a modified version which as he said change/delete some rules from the original and it's base on the version 2 years ago. I've transalted a new version base on current style guide of es5 version.

The current Chinese version translated by adamlu is a modified version which as he said change/delete some rules from the original and it's base on the version 2 years ago. I transalte a new version base on current style guide of es5 version.
@hethune

hethune commented Jun 1, 2015

Copy link
Copy Markdown

@hshoff ?

@panxianhai

Copy link
Copy Markdown

@sivan is it better to translate a complete version?

@1ternal

1ternal commented Jun 11, 2015

Copy link
Copy Markdown

I agree with @sivan, it's better not to modify the origin.

@lijunle

lijunle commented Jun 11, 2015

Copy link
Copy Markdown

I think @sivan 's translation is newer and better.

For the space rule,

The old one still suggests 4 spaces indentation,
The new one suggests 2 spaces indentation.

@hshoff

hshoff commented Jun 17, 2015

Copy link
Copy Markdown
Member

@sivan thanks for newer/better translation! 🍻

hshoff added a commit that referenced this pull request Jun 17, 2015
Change Chinese Translation to a newer version
@hshoff
hshoff merged commit a872822 into airbnb:master Jun 17, 2015
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.

6 participants