Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

make Korean docs up-to-date #1032

Merged
merged 1 commit into from
Feb 9, 2017
Merged

Conversation

outsideris
Copy link
Contributor

follow changed in English and #1022

@outsideris
Copy link
Contributor Author

@nodejs/nodejs-ko

The website working group's purpose is to build and maintain a public
website for the `Node.js` project.
The Website Working Group's purpose is to build and maintain a public
website for the Node.js project.
Copy link
Contributor

Choose a reason for hiding this comment

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

This change (`Node.js` → Node.js) is not applied to the translation.

* Foster and enable a community of translators.
Responsibilities include:
* Developing and maintaining a build and automation system for nodejs.org.
* Ensuring the site is regularly updated with changes made to Node.js, like
Copy link
Contributor

Choose a reason for hiding this comment

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

Also please change `nodejs.org` to nodejs.org, `Node.js` to Node.js in translation.

* Node.js 이슈 트래커에서 스트림 관련 이슈를 처리합니다.
* Node.js 프로젝트 내 스트림 문서를 작성하고 수정합니다.
* Node.js 프로젝트 내 스트림 하위 클래스의 변경사항을 리뷰합니다.
* 스트림 변경사항을 Node.js 프로젝트에서 이 프로젝트로 리다이렉트합니다.
* Node.js 내 스트림 프로바이더의 구현체를 지원합니다.
* 읽기 가능한 스트림의 버전이 Node.js에 포함되도록 권장합니다.
* `읽기 가능한 스트림`의 버전이 Node.js에 포함되도록 권장합니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Use 'readable-stream' as is instead of '읽기 가능한 스트림', as it's a package name.


트레이싱 워킹 그룹의 목적은 Node.js로 작성된 소프트웨어의 투명성을 증가시키는 것입니다.
Diagnostics 워킹 그룹의 목적은 Node.js 도구와 JavaScript VM에서 사용할 수 있는
광범위하고 확장성 진단 인터페이스를 만들고 문서화하는 것입니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

How about '광범위한'?

* 추적 및 디버깅 생태계를 육성합니다.
* `v8_inspector`를 Node.js에 통합하도록 V8과 협업합니다.
* `trace_event`를 Node.js에 통합하도록 V8과 협업합니다.
* `async_wrap`과 `async_hooks`을 개량할 수 있도록 Core 팀과 협업합니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe '`async_hooks`를' here.

@@ -633,8 +610,7 @@ Its responsibilities are:
The Node.js Testing Working Group's purpose is to extend and improve testing of
the Node.js source code.

It's responsibilities are:

Copy link
Contributor

Choose a reason for hiding this comment

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

Remove this empty line in translation.

@@ -251,15 +251,17 @@ Run as root on RHEL, CentOS or Fedora, for Node.js v4 LTS Argon:
CentOS 5)의 Node.js 패키지도 사용할 수 있습니다. 설치되어 있지 않다면 설정 스크립트가 확인하고
안내할 것입니다.

Node.js v6 LTS를 설치하려면 RHEL, CentOS or Fedora에서 root로 실행하세요.
Copy link
Contributor

Choose a reason for hiding this comment

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

RHEL, CentOS or Fedora → RHEL, CentOS, Fedora

<!--
```bash
curl --silent --location https://rpm.nodesource.com/setup_4.x | bash -
curl --silent --location https://rpm.nodesource.com/setup_6.x | bash -
Copy link
Contributor

Choose a reason for hiding this comment

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

Not applied in translation.

```

Alternatively for Node.js v6:
Alternatively for Node.js v7:
Copy link
Contributor

Choose a reason for hiding this comment

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

Not applied in translation.


```bash
curl --silent --location https://rpm.nodesource.com/setup_6.x | bash -
curl --silent --location https://rpm.nodesource.com/setup_7.x | bash -
Copy link
Contributor

Choose a reason for hiding this comment

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

Not applied in translation.

@outsideris
Copy link
Contributor Author

I updated as feedback. Thank you.

@outsideris
Copy link
Contributor Author

I rebased it on master to fix the build failure.

@fhemberger
Copy link
Contributor

@outsideris Hi, could you please rebase it again and fix the conflict? Then I'd be happy to merge it.

@outsideris
Copy link
Contributor Author

@fhemberger Sorry, I forgot it. I will address it.

@sotayamashita
Copy link
Contributor

sotayamashita commented Feb 4, 2017

@outsideris I think you should follow the commit message in TRANSLATION.md

For example:

nodejs-ko: up-to-date

@outsideris
Copy link
Contributor Author

@sotayamashita Oops! I didn't know the rule. I will fix commit messages in this PR.

@outsideris
Copy link
Contributor Author

@fhemberger @sotayamashita I rebased it and fixed the commit message.

@nodejs/nodejs-ko Please review this. I made this pull request about 2 months ago. In this time, I just rebase it onto latest matser branch.

* 모든 기능과 표준 준수 (표준: ECMA, 유니코드...)
* 트래커에 올라온 세계화(Globalization)와 국제화(Internationalization) 이슈를 지원합니다.
* 가이드라인과 권장 사례를 만듭니다.
* 기존의 `Intl` 구현체를 정체합니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you mean '정제합니다'?

-->

## .dockerignore 파일
`Dockerfile`과 같은 디렉터리에 `.dockerignore` 파일을 다음 내용을 만드세요.
Copy link
Contributor

Choose a reason for hiding this comment

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

  • Please insert a new line above this line.
  • 다음 내용을 → 다음 내용으로

@outsideris
Copy link
Contributor Author

I updated it. @yous Thank you for review.

@fhemberger fhemberger merged commit 87233c0 into nodejs:master Feb 9, 2017
@fhemberger
Copy link
Contributor

Thank you!

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.

5 participants