Skip to content
This repository was archived by the owner on May 7, 2022. It is now read-only.

Add Korean translations#28

Merged
mikeal merged 1 commit intonodejs:gh-pagesfrom
marocchino:korean
Mar 4, 2015
Merged

Add Korean translations#28
mikeal merged 1 commit intonodejs:gh-pagesfrom
marocchino:korean

Conversation

@marocchino
Copy link
Contributor

No description provided.

@bnb
Copy link

bnb commented Mar 1, 2015

@marocchino Can you get someone from @iojs/iojs-ko to +1 this?

@marocchino
Copy link
Contributor Author

@bnb It still working, I will request review later. 🙇

@marocchino marocchino changed the title Add Korean translations (wip) Add Korean translations Mar 1, 2015
@marocchino
Copy link
Contributor Author

Okey, done. @iojs/iojs-ko , @yous review please.

index.html Outdated
Copy link

Choose a reason for hiding this comment

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

Actually 'Stability Policy' heading is <h2> and 'Long Term Support' heading is <h3>. Is there any reason to change this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I thought It's miss duplicated from L94 and it is more reasonable match with slide title.
Maybe, It needs to confirm from admin.

Copy link

Choose a reason for hiding this comment

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

To be fair, you're both technically correct (not, however, for the reason you assumed, @marocchino), as this slide pulls from both the Stability Policy and the Long Term Support sections of the ROADMAP.md on iojs/io.js. I would say leave it as it was (Stability Policy), for two reasons:

  • Two of the three points in this slide are from the Stability Policy section of the ROADMAP.md, and one is from the Long Term Support section
  • The Stability is directly above the LTS section in ROADMAP.md, so it makes sense in terms of order to credit it to Stability, as the reader will naturally flow from Stability to LTS.

That said, this is something to consider changing to make more clear and/or 100% accurate. Neither of you are wrong. But I would say the best course of action is to leave the Stability Policy text as is. /ping @mikeal

ko/index.html Outdated
Copy link

Choose a reason for hiding this comment

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

컨트리뷰터 → 기여자
active → 활발한

이미 node.js의 모든 기록을 뛰어넘는 더 많은 active 개발자를 모집했다.

For the 'in the first few months', maybe we should change the term '이미' to '불과 몇 개월 만에'. We could say '불과 몇 개월 만에 node.js 역사상 가장 많은 수의 활발한 개발자를 모집했다.'.

ko/index.html Outdated
Copy link

Choose a reason for hiding this comment

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

We can use '부버전' used in http://semver.org/lang/ko/, otherwise just '마이너 버전' can be used.

어떤 API 추가도 minor 버전을 증가시킨다.

Seeing http://semver.org/lang/ko/, we could say 'API를 추가할 때는 minor 버전을 올린다.'. If you want to emphasize 'any', '어떤 API를 추가하더라도 minor 버전을 올린다.' also can be used, but I like the former one.

@marocchino marocchino force-pushed the korean branch 5 times, most recently from ece30c8 to 67a38bd Compare March 3, 2015 15:55
@marocchino
Copy link
Contributor Author

Reviews in 71bcf2b also appled.

@outsideris
Copy link

My review is done. 👍

@yous
Copy link

yous commented Mar 3, 2015

Also I'm done.

@marocchino
Copy link
Contributor Author

@bnb Review is over. Just check the "summary" part, please. Should I revert it?

@marocchino
Copy link
Contributor Author

@mikeal ping!

@bnb
Copy link

bnb commented Mar 4, 2015

@marocchino In the future, leave changes to the English files in their own PRs, and translation changes in their own PRs. It's much easier to track the changes that way.

@marocchino
Copy link
Contributor Author

@bnb Okey. I reverted it.
Thank you.

@mikeal
Copy link
Contributor

mikeal commented Mar 4, 2015

Is this ready? Can you add this to the list of to the list of available translations in the index.

@marocchino
Copy link
Contributor Author

@mikeal did it. :)

mikeal added a commit that referenced this pull request Mar 4, 2015
@mikeal mikeal merged commit e56191c into nodejs:gh-pages Mar 4, 2015
@marocchino marocchino deleted the korean branch March 4, 2015 22:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants