- 
          
 - 
                Notifications
    
You must be signed in to change notification settings  - Fork 6.4k
 
Translate "docs" menu as Korean #904
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
Conversation
| 
           저희 저장소가 아니라 메뉴별로 하다보니 PR에 내용이 꽤 많습니다. 시간나실 때 천천히 봐주세요.  | 
    
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
locale/ko/docs/meta/topics/dependencies.md is not translated. consider remove this file.
I'm done.
| * All **shipping** features, which V8 considers stable, are turned **on by default on Node.js** and do **NOT** require any kind of runtime flag. | ||
| * **Staged** features, which are almost-completed features that are not considered stable by the V8 team, require a runtime flag: `--harmony`. | ||
| * **In progress** features can be activated individually by their respective harmony flag, although this is highly discouraged unless for testing purposes. Note: these flags are exposed by V8 and will potentially change without any deprecation notice. | ||
| --> | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It will be safer add blank line around comment blocks. (unless you checked all pages.)
Some parser dose not handle this blocks properly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with you even though I checked all pages.
        
          
                locale/ko/docs/es6.md
              
                Outdated
          
        
      | * **Staged** 기능은 거의 완성되었지만, V8 팀이 안정적이라고 간주하지 않은 기능으로 | ||
| `--harmony` 런타임 플래그가 필요합니다. | ||
| * **In progress** 기능은 각 하모니 플래그로 개별적으로 활성화할 수 있습니다. 테스트 목적이 | ||
| 아니라면 활성화하지 않기를 강력하게 권장한다. Note: 이 플래그는 V8에서 제공한 것으로 | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(for consistency)
권장한다 -> 권장합니다
        
          
                locale/ko/docs/es6.md
              
                Outdated
          
        
      | --> | ||
| ## Node.js 버전에 어떤 기능이 기본적으로 포함되나요? | ||
| 
               | 
          ||
| [node.green](http://node.green) 웹사이트에서 Node.js의 다양한 버전에서 지원하는 | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This sentence has two 에서. I think it is confuse to someone.
How about this:
kangax의 호환성 표를 기반으로 만들어진 node.green 웹사이트에서 어떤 Node.js가 어떤 ECMAScript 기능을 지원하는지 쉽게..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이쪽 저장소지만 한글로 쓰셔도 될꺼라고 생각합니다. 한국어 리뷰라서요...(왠지 힘드시지 않을까 싶어서.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
..음.. 그럴까요?;
        
          
                locale/ko/docs/es6.md
              
                Outdated
          
        
      | 
               | 
          ||
| `--v8-options` 인자로 각 Node.js 릴리스에서 모든 *in progress* 기능의 리스트를 볼 수 | ||
| 있습니다. 이 기능들은 완성되지 않았고 V8에서 제대로 돌아가지 않을 수도 있으므로 이 기능을 사용할 때는 | ||
| 위험을 고려해야 함을 명심하세요. | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
위험을 고려해야 -> 위험을 감수해야
| <!-- | ||
| ## What is open source governance? | ||
| 
               | 
          ||
| Open source governance advocates the application of the philosophies of the open source and open content movements in order to enable any interested party to add to the creation of the end product, as with a wiki document. Legislation is democratically opened to the general citizenry, employing their collective wisdom to benefit the decision-making process and improve democracy. [[source]](https://en.wikipedia.org/wiki/Open-source_governance) | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the democracy noun here? -3-
Weird.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
의미는 "더 민주적으로 된다"정도로 이해했는데 좀더 좋은 문장이 있을까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네 의미상 부사일거라 생각했는데, democracy는 명사밖에 없네요. 이상하긴한데 변역은 틀리지 않은것 같습니다.
| 비교하는지 볼 수 있다면 파일시스템을 탐구해야 합니다.) 예를 들어 대소문자 비구별은 단순한 | ||
| `toLowerCase()` 비교보다 훨씬 복잡합니다. 사실 `toUpperCase()`가 `toLowerCase()`보다 | ||
| 보통 더 좋습니다.(`toLowerCase()`가 특정 외국어 문자를 다르게 다루기 때문입니다.) 하지만 모든 | ||
| 파일시스템은 자신만의 대소문자 비교 테이블을 가지고 있으므로 더 좋은 방법을 쓰더라도 파일시스템을 | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
더 좋은 방법을 쓰더라도 파일시스템을 탐구해야 합니다. -> 파일시스템을 탐구하는 것이 좋습니다.
        
          
                locale/ko/docs/index.md
              
                Outdated
          
        
      | --> | ||
| # 문서에 관해서 | ||
| 
               | 
          ||
| 이 웹사이트에는 세 가지 종류의 문서가 있다. | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
있다 -> 있습니다
        
          
                locale/ko/docs/index.md
              
                Outdated
          
        
      | --> | ||
| ### API 레퍼런스 문서 | ||
| 
               | 
          ||
| [API 레퍼런스 문서](/api/)에는 Node.js의 함수나 객체에 대한 자세한 정보가 있다. | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
있다 -> 있습니다
        
          
                locale/ko/docs/index.md
              
                Outdated
          
        
      | 
               | 
          ||
| [API 레퍼런스 문서](/api/)에는 Node.js의 함수나 객체에 대한 자세한 정보가 있다. | ||
| 이 문서에서 메서드가 어떤 인자를 받고 어떤 값을 반환하는지 해당 메서드와 관련된 에러에는 어떤 것이 | ||
| 있는지를 알려준다. 다양한 Node.js에서 어떤 메서드를 사용할 수 있는지도 알려준다. | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
알려준다 -> 알려줍니다
        
          
                locale/ko/docs/index.md
              
                Outdated
          
        
      | ### 자주 묻는 질문 | ||
| 
               | 
          ||
| [FAQ](/ko/docs/faq/)에서는 Node.js에 기여하는 방법, 행동강령과 거버넌스 모델, | ||
| GitHub과 IRC에서 연락을 취하는 방법, 선별된 이슈로 돕는 방법 등을 다룬다. | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
다룬다 -> 다룹니다
| @@ -0,0 +1,102 @@ | |||
| --- | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아 이 파일은 포함된지 모르고 있었네요. 확인하고 빼던지 추가로 번역하든지 하겠습니다.
        
          
                locale/ko/docs/es6.md
              
                Outdated
          
        
      | * **Staged** 기능은 거의 완성되었지만, V8 팀이 안정적이라고 간주하지 않은 기능으로 | ||
| `--harmony` 런타임 플래그가 필요합니다. | ||
| * **In progress** 기능은 각 하모니 플래그로 개별적으로 활성화할 수 있습니다. 테스트 목적이 | ||
| 아니라면 활성화하지 않기를 강력하게 권장한다. Note: 이 플래그는 V8에서 제공한 것으로 | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
권장한다 > 권장합니다.
Note: > 주의:
        
          
                locale/ko/docs/es6.md
              
                Outdated
          
        
      | --> | ||
| ## 어떤 기능이 진행 중입니까? | ||
| 
               | 
          ||
| V8 엔진에 계속해서 새로운 기능이 들어가고 있습니다. 대략 말하자면 시기는 알 수 없어도 | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
들어가고 > 추가되고
로 변경하는 편이 더 자연스럽게 들릴 듯 합니다.
        
          
                locale/ko/docs/es6.md
              
                Outdated
          
        
      | ## 어떤 기능이 진행 중입니까? | ||
| 
               | 
          ||
| V8 엔진에 계속해서 새로운 기능이 들어가고 있습니다. 대략 말하자면 시기는 알 수 없어도 | ||
| 미래의 Node.js 릴리스에는 이 기능들이 들어온다고 할 수 있습니다. | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
정확한 시기는 알 수 없겠지만 앞으로 릴리스 할 Node.js에는 V8에 추가된 새 기능이 대체로 포함될 것입니다.
        
          
                locale/ko/docs/faq.md
              
                Outdated
          
        
      | 지키고 [오픈 거버넌스](https://github.com/nodejs/node/blob/master/GOVERNANCE.md#readme) | ||
| 모델 하에 기여, 릴리스, 기여자를 운영합니다. | ||
| 
               | 
          ||
| [GitHub에 공개적으로 논의](https://github.com/nodejs/node/issues)를 하고 있고 우리는 | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
말이 조금 어색해서 살짝 의역해보았습니다.
우선은 GitHub에서 공개적으로 이뤄지는 논의에 참여해보는 것도 좋습니다. 우리는 여러분의 피드백을 듣고 싶습니다.
| 이 문서의 목적은 Node.js HTTP 처리 과정을 잘 이해하게 하는 것입니다. 언어나 프로그래밍 환경과 | ||
| 관계없이 상식선에서 HTTP 요청이 어떻게 동작하는지는 알고 있어야 합니다. Node.js의 | ||
| [`EventEmitters`][]와 [`Streams`][]에도 어느 정도 익숙해야 합니다. 만약 익숙하지 않다면 | ||
| 이에 대한 API 문서를 훑어볼 필요가 있습니다. | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
익숙하지 않다면 관련 API 문서를 미리 훑어보는 편이 좋습니다.
- 만약과 ~면은 중복되는 표현이라 하나만 사용했습니다.
 - it's worth는 반드시 해야한다는 필요보다는 권장 수준으로 보는 게 좋을 것 같습니다.
 - 이 문서를 읽기 전에 먼저 훑어보라는 뜻이라 문장이 자연스럽도록 "미리"를 임의로 추가했습니다만, 불필요하다 생각하면 빼셔도 괜찮습니다.
 
| ## 서버 생성 | ||
| 
               | 
          ||
| 모든 node 웹 서버 애플리케이션은 웹 서버 객체를 만들어야 합니다. | ||
| 이는 [`createServer`][]를 이용합니다. | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이는 > 이 때
| too, so consult the [API reference][]. | ||
| --> | ||
| HTTP 요청이 서버에 오면 node가 트랜잭션을 다루려고 `request`과 `response` 객체로 | ||
| 요청 핸들러 함수를 호출합니다. 곧 이 객체를 사용해 볼 것입니다. | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
객체로 함수를 호출한다는 표현이 애매합니다.
객체와 함께 또는 객체를 전달하며라고 바뀌어야 할 것 같습니다.
| --> | ||
| ## 메서드, URL, 헤더 | ||
| 
               | 
          ||
| 아마 요청을 처리할 때 처음에는 메서드와 URL을 살펴보고자 할 것이므로 이에 적절한 작업을 할 수 있습니다. | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
요청을 처리할 때, 우선은 메서드와 URL을 확인한 후 이와 관련된 적절한 작업을 실행하려고 할 것입니다.
| ## 메서드, URL, 헤더 | ||
| 
               | 
          ||
| 아마 요청을 처리할 때 처음에는 메서드와 URL을 살펴보고자 할 것이므로 이에 적절한 작업을 할 수 있습니다. | ||
| Node가 `request` 객체에 유용한 프로퍼티를 넣어두었으므로 비교적 쉽게 이 작업을 할 수 있습니다. | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
비교적 쉽게 이 작업을 할 수 있습니다. > 이 작업은 비교적 쉽게 할 수 있습니다.
로 바꾸는 편이 더 읽기 나은 것 같습니다.
| ``` | ||
| --> | ||
| 여기서 `method`는 항상 일반적인 HTTP 메서드/동사가 될 것입니다. `url`은 서버, 프로토콜, | ||
| 포트를 제외한 전체 URL입니다. 즉, URL의 세 번째 슬래시부터 모든 것을 담고 있습니다. | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
url은 전체 URL에서 서버, 프로토콜, 포트를 제외한 것으로, 세 번째 슬래시 이후의 나머지 전부라고 볼 수 있습니다.
| 
           I've applied the reviews that you provided. I also translated   | 
    
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
response 객체는 WritableStream이므로 클라이언트로 보내는 응답 바디는 일반적인 스트림 메서드를 사용해서 작성합니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
위의 예제를 간략하게 바꾸어 간단한 에코 서버를 만들어보겠습니다. 에코 서버는 요청받은 데이터를 그대로 응답으로 돌려보내는 서버입니다. 앞에서 했던 것처럼 요청 스트림에서 데이터를 가져와 응답 스트림에 쓰기만 하면 됩니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
목차와 타이틀이 일치했으면 좋겠습니다.
-- [Libraries](#a-id-libraries-a)
+- [라이브러리](#libraries)
-## <a id="Libraries">라이브러리</a>
+## <!--Libraries-->라이브러리There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오! 저렇게도 앵커가 되나요? 앵커가 안되서 저렇게 했던거거든요. 테스트하고 업데이트하겠습니다.
ef2c35c    to
    abb43e5      
    Compare
  
    | 
           @nodejs/nodejs-ko Sorry for being late. I changed it as the feedbacks. Please leave +1 or mark as approved if you don't have any concerns.  | 
    
        
          
                locale/ko/docs/es6.md
              
                Outdated
          
        
      There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
--es_staging와 → --es_staging과
        
          
                locale/ko/docs/es6.md
              
                Outdated
          
        
      There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아직 안정적이라고 간주하지 않은 것입니다
How about '아직 안정적이라고 볼 수는 없습니다'?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
메서드 → 메소드
Also there are several '메서드's elsewhere.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
덮어써 지거나 → 덮어씌워지거나
Maybe '쉼표' instead of '콤마'? I'm okay with both.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[body][]같은 → [body][] 같은
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
20 요청을 → 20개의 요청을
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
베스트 프렉티스를 → 모범 예제를, 좋은 예제를
        
          
                locale/ko/docs/index.md
              
                Outdated
          
        
      There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
메서드 → 메소드
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넌블락킹 → 논블로킹
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
만들어 진 → 만들어진
abb43e5    to
    fd086f7      
    Compare
  
    | 
           I updated it as the reviews.  | 
    
| 
           i18n looks fine to me. 👍  | 
    
| 
           And, 👍 for WG. It will be fine. :3  | 
    
| 
           🎉  | 
    
* upstream/master: outreachy: add myles' project add good first contributions to the outreachy page precede email addresses with mailto (nodejs#935) Translate "docs" menu as Korean (nodejs#904) nodejs-ja: Add about/index.md (nodejs#933) Revert "Blog: fix CVE ID for writeHead() bug / CVE-2016-5326" Blog: fix CVE ID for writeHead() bug / CVE-2016-5326 Add Education initiatives and Outreachy to Foundation section of website (nodejs#929)
Please review this @nodejs/nodejs-ko