-
Notifications
You must be signed in to change notification settings - Fork 68
Translate Node v11.4.0, v11.5.0, v11.6.0 #791
Conversation
* `console` 함수가 스펙에 정의된 대로 심볼을 처리합니다. https://github.com/nodejs/node/pull/23708 | ||
* `depth` 검사의 기본값이 다시 2로 돌아갑니다. https://github.com/nodejs/node/pull/24326 | ||
* **dgram, net**: | ||
* `net`과 `dgram`에 ipv60nly 옵션이 추가됩니다. https://github.com/nodejs/node/pull/23798 |
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.
ipv6Only
가 ipv60nly
로 쓰인 것 같습니다.
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.
아 감사합니다 😢😢😢😢😢😢
* 이제 `pathToFileURL()`이 LF, CR, TAB을 지원합니다. https://github.com/nodejs/node/pull/23720 | ||
* **Windows**: | ||
* 더 이상 Boxstarter를 사용하여 도구들이 설치되지 않습니다. https://github.com/nodejs/node/pull/24677 | ||
* install-tools 스크립트 또는 이제 dist에 포함되어 있습니다. https://github.com/nodejs/node/pull/24233 |
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.
PR의 내용을 보면 원문이 약간 이상한 것 같고 install-tools 스크립트가 이제 배포판에 포함되었습니다.
정도로도 괜찮을 것 같습니다.
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.
그러게요. 'or' 대신 'are'이라고 하면 좀 말이 되는 것 같습니다.
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.
are이라고 하면 outsideris님의 번역이 맞다고 생각합니다. 그렇게 가정하고 제안해 주신 대로 수정할게요.
* **cli**: | ||
- --max-http-header-size 플래그를 추가했습니다. (cjihrig) [#24811](https://github.com/nodejs/node/pull/24811) | ||
* **crypto**: | ||
- 인증서를 항상 공개키로 허용합니다. (Tobias Nießen) [#24234](https://github.com/nodejs/node/pull/24234) |
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.
네 저건 번역을 잘못 했네요.. 지적해주셔서 감사합니다
- --max-http-header-size 플래그를 추가했습니다. (cjihrig) [#24811](https://github.com/nodejs/node/pull/24811) | ||
* **crypto**: | ||
- 인증서를 항상 공개키로 허용합니다. (Tobias Nießen) [#24234](https://github.com/nodejs/node/pull/24234) | ||
- key object API가 추가되었습니다. (Tobias Nießen) [#24234](https://github.com/nodejs/node/pull/24234) |
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.
PR을 보면 crypto의 Key 객체와 관련된 API 같아서 key 객체 관련 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.
PR의 커밋들을 보면 keyobject
라는 이름이 들어간 클래스들이 있어서 위와같이 번역했는데, 자세히 보니 outsideris님의 지적이 맞겠다고 생각되네요. 제안해 주신 대로 수정하겠습니다.
* 이제 `pathToFileURL()`이 LF, CR, TAB을 지원합니다. https://github.com/nodejs/node/pull/23720 | ||
* **Windows**: | ||
* 더 이상 Boxstarter를 사용하여 도구들이 설치되지 않습니다. https://github.com/nodejs/node/pull/24677 | ||
* install-tools 스크립트 또는 이제 dist에 포함되어 있습니다. https://github.com/nodejs/node/pull/24233 |
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.
그러게요. 'or' 대신 'are'이라고 하면 좀 말이 되는 것 같습니다.
Co-Authored-By: Fleta <yasoueisin@gmail.com>
리뷰해 주신 내용들은 전부 확인했습니다. 오늘 저녁에 전부 반영하고 수정커밋 할게요. 감사합니다 !! |
업데이트 확인했습니다. 아직 적용 안 된 부분이 몇 군데 있는 것 같습니다. |
확인해 주셔서 감사합니다. 제가 머지를 잘못해서 v11.4.0에 수정사항 적용이 제대로 안 되어 있었습니다. v11.4.0 v11.5.0 v11.6.0 |
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.
@yous 님이 얘기하신 부분 말고는 전 괜찮습니다.
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.
감사합니다!
close #779
close #782
close #788
전부 11버전 업데이트 뉴스여서 한번에 진행했습니다.