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

Translate 2015-03-20-weekly #41

Merged
merged 1 commit into from
Mar 28, 2015
Merged

Conversation

outsideris
Copy link
Contributor

아직 Medium에 글은 안올라왔지만 리뷰도 하고 해야 하니 일단 올립니다.
원문에 대한 링크랑 정보는 Medium에 글이 올라오면 수정하겠습니다.

* **node**: 시작할 때 모듈을 미리 로딩하기 위해 새로운 커맨드라인 옵션 `-r`나 `--require`를 사용할 수 있습니다. (Ali Ijaz Sheikh) [#881](https://github.com/iojs/io.js/pull/881)
* **querystring**: `parse()`와 `stringify()`가 더 빨라졌습니다. (Brian White) [#847](https://github.com/iojs/io.js/pull/847)
* **http**: [joyent/node#9048](https://github.com/joyent/node/pull/9048)에 따라 Node.js v0.12에서 변경된 내용과 맞추려고 `http.ClientRequest#flush()` 메서드를 폐기하고 `http.ClientRequest#flushHeaders()`로 대체했습니다. (Yosuke Furukawa) [#1156](https://github.com/iojs/io.js/pull/1156)
* **net**: [joyent/node#9268](https://github.com/joyent/node/pull/9268)에서 `net.connect()`가 받는 옵션과 같게 하려고 `server.listen()`가 `port`를 `String` 옵션으로 받을 수 있게 되었습니다. (예시: `{ port: "1234" }`) (Ben Noordhuis) [#1116](https://github.com/iojs/io.js/pull/1116)
Copy link
Contributor

Choose a reason for hiding this comment

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

server.listen()가 -> server.listen()

@yous
Copy link
Contributor

yous commented Mar 23, 2015

저는 일단 다 봤습니다.


* **path**: `path.resolve()` [#1153](https://github.com/iojs/io.js/pull/1153)의 새로운 타입 검사는
실무에서 발생하는 일부 경계조건(특히 `path.dirname(undefined)`)은 다루지 않습니다. 타임 검사는 `path.dirname()`, `path.basename()`, `path.extname()` 때문에 느슨해졌습니다. (Colin Ihrig) [#1216](https://github.com/iojs/io.js/pull/1216).
* **querystring**: `querystring.parse()`와 `querystring.stringify()`의 내부 최적화([#847](https://github.com/iojs/io.js/pull/847))에서 `querystring.escape()`이 `Number` 리터럴을 적절하게 변환하지 못하는 문제가 있었습니다.([#1208](https://github.com/iojs/io.js/issues/1208)) 이는 테스트 슈트에서 발견하지 못한 문제이지만 지금은 버그와 테스트가 수정되었습니다. (Jeremiah Senkpiel) [#1213](https://github.com/iojs/io.js/pull/1213).
Copy link
Contributor Author

Choose a reason for hiding this comment

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

버그에 대해서 @yous 님 말씀이 맞는것 같아서 이렇게 문장을 풀어봤습니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

여기에는 "가" 가 자연스러울것 같긴한데, 한번 더 생각해보니 escape을 이스케입이라 읽을수도 있겠다는 생각이 드네요. 어느 쪽이 나을까요?

querystring.escape()

Copy link
Contributor

Choose a reason for hiding this comment

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

한글로 표기할 땐 '이스케이프'가 될 것이라 '가'를 쓰는 게 좋을 것 같습니다.

@outsideris
Copy link
Contributor Author

리뷰 받은 부분 적용했습니다.

* express.js가 io.js 지원을 추가했습니다.
* 지난 두 주 사이에 Joyent의 하드웨어에 접근 권한을 얻고 V8에 패치를 올려서 io.js를 빌드할 수 있게 되었습니다. [SmartOS](https://github.com/iojs/build/pull/64)와 [FreeBSD](https://github.com/iojs/io.js/pull/1167)에서 테스트를 통과하도록 작업한 결과 이틀 전부터 테스트를 통과하게 되었습니다. 이는 빌드 팀과 [Johan Bergström](https://github.com/jbergstroem)의 놀라운 작업 덕분입니다.
* [Petka Antonov](https://github.com/petkaantonov)가 실험적으로 io.js에서 워커 구현체를 제안하고 있습니다. [여기서](https://github.com/iojs/io.js/pull/1159) 이에 대한 논의에 참여할 수 있습니다.
* io.js는 openssl을 `1.0.1m`로 [업그레이드](https://github.com/iojs/io.js/pull/1206)했습니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

여기는 "로"보다는 "으로"가 자연스러울 것 같습니다.

@outsideris
Copy link
Contributor Author

수정했습니다.

@marocchino
Copy link
Contributor

👍


* **path**: `path.resolve()` [#1153](https://github.com/iojs/io.js/pull/1153)의 새로운 타입 검사는
실무에서 발생하는 일부 경계조건(특히 `path.dirname(undefined)`)은 다루지 않습니다. 타임 검사는 `path.dirname()`, `path.basename()`, `path.extname()` 때문에 느슨해졌습니다. (Colin Ihrig) [#1216](https://github.com/iojs/io.js/pull/1216).
* **querystring**: `querystring.parse()`와 `querystring.stringify()`의 내부 최적화([#847](https://github.com/iojs/io.js/pull/847))에서 `querystring.escape()`가 `Number` 리터럴을 적절하게 변환하지 못하는 문제가 있었습니다.([#1208](https://github.com/iojs/io.js/issues/1208)) 이는 테스트 슈트에서 발견하지 못한 문제이지만 지금은 버그와 테스트가 수정되었습니다. (Jeremiah Senkpiel) [#1213](https://github.com/iojs/io.js/pull/1213).
Copy link
Contributor

Choose a reason for hiding this comment

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

'test suite'의 suite는 '스위트'라고 표기하는 것 같습니다. http://krdic.naver.com/detail.nhn?docid=23168600#FGN17892

그런데 '테스트 스위트'는 어색할 것 같으니 '테스트 모음'이나 '테스트 묶음', '테스트 세트' 등이 적절할 것 같습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

저는 tdd에서 사용하는 용어고 슈트라고 많이 표기해서 괜찮다고 보는 편입니다.
슈트라는 표기가 어색하다면 이 문맥에서는 그냥 "테스트에서는 발견하지 못한"이라고 해도 우리 나라 말에서는 괜찮을것 같기도 하고요.

Copy link
Contributor

Choose a reason for hiding this comment

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

네, 그냥 테스트라고 쓰는 것도 좋을 것 같습니다. 또한 앞 문장의 '문제입니다'와 '문제이지만'이 반복되고 있으니 '발견하지 못한 것이지만', '발견하지 못했지만' 등으로도 적을 수 있을 것 같습니다.

@outsideris
Copy link
Contributor Author

리뷰 내용 적용했습니다.

* 대화형 셸에서 서러게이트 페어(Surrogate pair)가 터미널을 정지시킬 수 있습니다. [#690](https://github.com/iojs/io.js/issues/690)
* io.js를 정적 라이브러리로 빌드할 수 없습니다. [#686](https://github.com/iojs/io.js/issues/686)
* `process.send()`는 문서에서 설명된 바와는 다르게 동기적이지 않습니다. 이 회귀는 1.0.2에서 발생했습니다. [#760](https://github.com/iojs/io.js/issues/760)에서 확인 가능하며 [#774](https://github.com/iojs/io.js/issues/774)에서 수정하고 있습니다.
* DNS 쿼리 중에 `dns.setServers()`를 호출하면 단언문 실패 때문에 크래시가 발생할 수 있습니다. [#894](https://github.com/iojs/io.js/issues/894)
Copy link
Contributor

Choose a reason for hiding this comment

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

weekly/_posts/2015-03-13-weekly.md 파일도 함께 수정해 주시면 감사하겠습니다.

@outsideris
Copy link
Contributor Author

"실패때문에" 부분 말씀이시죠? 같이 수정했습니다.

@yous
Copy link
Contributor

yous commented Mar 26, 2015

😃

@outsideris
Copy link
Contributor Author

다른 분들 이견 없으시면 머지할까요?

@yous
Copy link
Contributor

yous commented Mar 28, 2015

좋습니다.

outsideris added a commit that referenced this pull request Mar 28, 2015
@outsideris outsideris merged commit 8cd4009 into nodejs:gh-pages Mar 28, 2015
@yous yous removed the in review label Mar 28, 2015
@outsideris outsideris deleted the weekly-03-20 branch April 1, 2015 14:33
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.

5 participants