Skip to content

#233: 1-js/01-getting-started/1-intro/article.md 2차 리뷰#234

Merged
Violet-Bora-Lee merged 1 commit intojavascript-tutorial:masterfrom
sojeongw:master
Oct 1, 2019
Merged

#233: 1-js/01-getting-started/1-intro/article.md 2차 리뷰#234
Violet-Bora-Lee merged 1 commit intojavascript-tutorial:masterfrom
sojeongw:master

Conversation

@sojeongw
Copy link
Contributor

@sojeongw sojeongw commented Sep 29, 2019

처음 튜토리얼을 읽는 입장에서 이해가 수월하도록 자잘한 부분을 수정해보았는데 어떠실지 모르겠습니다.
보라님이 작성/리뷰하신 기존 글을 감히(?) 제가 수정하게 되니 겁도 나네요. 잘못한 점이 있다면 피드백 마구마구 부탁드립니다 :)

  1. 코드네임: 원본에서 term으로 되어있어 '용어'로 변경했습니다. 처음 접하는 사람에게도 '용어'가 더 쉽게 이해될 것 같다는 생각이 들었습니다.
  2. 문장에서 단어가 중복되는 것을 피하려고 했습니다.
  3. 한 번에 물 흐르듯 읽히지 않는 문장을 수정했습니다.
  4. 형태(?)를 맞추려고 했습니다. 예를 들어, '읽거나 쓰기'는 동사와 명사의 결합이므로 하나로 통일하려고 했습니다.
  5. 영어 튜토리얼과 다른 부분이 종종 있던데 그냥 번역 과정에서 추가하신 건지 영어 튜토리얼이 업데이트 된 것인지 모르겠네요.

@sojeongw
Copy link
Contributor Author

앗 다른 내용의 커밋인데 같은 PR로 합쳐지는군요..!

- IE는 버전에 따라 "Trident"나 "Chakra"라 불리는 엔진을 사용합니다. "ChakraCore"는 Microsoft Edge에 사용되며, "SquirrelFish"는 Safari에 사용됩니다.

개발 관련 글에서 종종 이 코드네임들이 언급되기 때문에 이를 기억해 두는 것이 좋습니다. 본 튜토리얼에서도 이 코드네임을 사용할 예정입니다. "X라는 기능은 V8에서만 지원합니다."라는 식으로 말이죠. 이런 문장을 만나면 Chrome과 Opera에서만 이 기능을 지원한다고 이해하시면 됩니다.
위의 용어는 개발 관련 글에서 종종 언급되기 때문에 기억해 두는 것이 좋습니다. 본 튜토리얼에서도 해당 용어를 사용할 예정입니다. "X라는 기능은 V8에서만 지원합니다."라는 식으로 말이죠. 이런 문장을 만나면 Chrome과 Opera에서만 이 기능을 지원한다고 이해하시면 됩니다.
Copy link
Member

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.

아하 그렇군요 그럼 코드 네임 그대로 하겠습니다!

3. 기계어로 전환된 코드가 실행됩니다. 기계어로 전환되었기 때문에 실행 속도가 빠릅니다.

엔진은 프로세스의 하위 단계 곳곳에서 최적화를 진행합니다. 심지어 컴파일이 끝나고 실행 중인 코드를 감시하면서, 이 코드로 흘러가는 데이터를 분석하고, 분석 결과를 토대로 기계어로 전환된 코드를 다시 최적화하기도 합니다. 이런 과정을 거치면 스크립트 실행 속도는 더욱 더 빨라집니다.
엔진은 프로세스 각 단계마다 최적화를 진행합니다. 심지어 컴파일이 끝나고 실행 중인 코드를 감시하면서, 이 코드로 흘러가는 데이터를 분석하고, 분석 결과를 토대로 기계어로 전환된 코드를 다시 최적화하기도 합니다. 이런 과정을 거치면 스크립트 실행 속도는 더욱 더 빨라집니다.
Copy link
Member

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.

저도 그렇게 이해했는데 말을 좀 더 쉽게 만들고 싶어서 억지로 줄인게 화근이 된 것 같네요. 다시 읽어보니 보라님이 쓰신 말이 더 맞는 것 같고 이해도 어렵지 않은 것 같습니다!

@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@Violet-Bora-Lee
Copy link
Member

Violet-Bora-Lee commented Sep 29, 2019

네 PR은 브랜치 단위로 이뤄지는 거라서요.
브랜치를 2개 만드셔서 PR을 2개로 쪼깨서 날려주세요.

이젠.. master브랜치에서만 작업하는걸 탈피해 봅시다! 👍 👍 👍

@Violet-Bora-Lee
Copy link
Member

감사합니다. 머지 진행하겠습니다
기여해주신 덕분에 훨씬 좋은 글이 되었네요 👍 👍 👍

@Violet-Bora-Lee Violet-Bora-Lee merged commit 3a0a5e0 into javascript-tutorial:master Oct 1, 2019
@Violet-Bora-Lee
Copy link
Member

다음엔 커밋 메시지만 신경써주세요.
커밋 메시지엔 경로는 잘 쓰지 않거든요.

@sojeongw
Copy link
Contributor Author

sojeongw commented Oct 1, 2019

앗 근데 아직 말씀하신 피드백을 반영하지 않은 커밋일텐데 머지가 된 것 같아요..!!

@Violet-Bora-Lee
Copy link
Member

Violet-Bora-Lee commented Oct 1, 2019

이럴수가... 제가 revert 할게요 다시 커밋 만들어주세요. Revert하고 코멘트 달게요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants