Skip to content
This repository has been archived by the owner on Mar 22, 2023. It is now read-only.

Translate Node v11.4.0, v11.5.0, v11.6.0 #791

Merged
merged 11 commits into from
Jan 10, 2019
Merged

Translate Node v11.4.0, v11.5.0, v11.6.0 #791

merged 11 commits into from
Jan 10, 2019

Conversation

Fleta
Copy link
Contributor

@Fleta Fleta commented Jan 4, 2019

close #779
close #782
close #788

전부 11버전 업데이트 뉴스여서 한번에 진행했습니다.

* `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
Copy link
Contributor

Choose a reason for hiding this comment

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

ipv6Onlyipv60nly로 쓰인 것 같습니다.

Copy link
Contributor Author

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
Copy link
Contributor

Choose a reason for hiding this comment

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

PR의 내용을 보면 원문이 약간 이상한 것 같고 install-tools 스크립트가 이제 배포판에 포함되었습니다. 정도로도 괜찮을 것 같습니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

그러게요. 'or' 대신 'are'이라고 하면 좀 말이 되는 것 같습니다.

Copy link
Contributor Author

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)
Copy link
Contributor

Choose a reason for hiding this comment

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

항상 공개키로 인증서를 받습니다.는 어떤가요?

Copy link
Contributor Author

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)
Copy link
Contributor

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가 추가되었습니다.는 어떤가요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

PR의 커밋들을 보면 keyobject라는 이름이 들어간 클래스들이 있어서 위와같이 번역했는데, 자세히 보니 outsideris님의 지적이 맞겠다고 생각되네요. 제안해 주신 대로 수정하겠습니다.

source/_posts/2018-12-07-release-v11.4.0.md Outdated Show resolved Hide resolved
source/_posts/2018-12-07-release-v11.4.0.md Outdated Show resolved Hide resolved
source/_posts/2018-12-07-release-v11.4.0.md Outdated Show resolved Hide resolved
* 이제 `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
Copy link
Contributor

Choose a reason for hiding this comment

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

그러게요. 'or' 대신 'are'이라고 하면 좀 말이 되는 것 같습니다.

source/_posts/2018-12-18-release-v11.5.0.md Outdated Show resolved Hide resolved
@Fleta
Copy link
Contributor Author

Fleta commented Jan 8, 2019

리뷰해 주신 내용들은 전부 확인했습니다. 오늘 저녁에 전부 반영하고 수정커밋 할게요. 감사합니다 !!

@yous
Copy link
Contributor

yous commented Jan 9, 2019

업데이트 확인했습니다. 아직 적용 안 된 부분이 몇 군데 있는 것 같습니다.

@Fleta
Copy link
Contributor Author

Fleta commented Jan 9, 2019

확인해 주셔서 감사합니다. 제가 머지를 잘못해서 v11.4.0에 수정사항 적용이 제대로 안 되어 있었습니다.
제가 받은 피드백의 상세 목록은 아래와 같습니다. 다시 확인하실 때 조금이라도 편하셨으면 좋겠습니다.

v11.4.0
ipv6Only
install-tools 스크립트가 이제 배포판에 포함 되었습니다
inspect의 depth 기본값이 ...
최저/최고 버전
런타임 플래그를 통해

v11.5.0
inspect에 getter 옵션을 추가했습니다

v11.6.0
항상 공개키로 인증서를 받습니다
key 객체 관련 API가 추가되었습니다.

Copy link
Contributor

@outsideris outsideris left a comment

Choose a reason for hiding this comment

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

@yous 님이 얘기하신 부분 말고는 전 괜찮습니다.

Copy link
Contributor

@yous yous left a comment

Choose a reason for hiding this comment

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

감사합니다!

@taggon taggon merged commit ed84afb into nodejs:master Jan 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Node v11.6.0 (Current) Node v11.5.0 (Current) Node v11.4.0 (Current)
4 participants