Skip to content
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

Translate 3 playground-examples files to ko #1082

Merged
merged 5 commits into from
Sep 18, 2020
Merged

Translate 3 playground-examples files to ko #1082

merged 5 commits into from
Sep 18, 2020

Conversation

seohyun0120
Copy link
Contributor

@seohyun0120 seohyun0120 commented Sep 13, 2020

en:

번역 리뷰 요청합니다: @yeonjuan @guyeol @dvlprsh

  1. 타언어 번역 폴더를 확인해보니 sections.json도 번역을 해둔 부분이 있어서 참고해서 번역했습니다.
  2. Export Modules From.ts 파일
    원문에는 re-export라고 되어있는 부분을 그대로 쓸까, 다시 export로 번역할지 고민하다가 후자를 선택했는데 의견 부탁드립니다. 어떤 곳(line 2) 에서는 원문 그대로가 매끄럽고 어떤 곳(line 6, 21)에서는 번역하는 것이 매끄러운 것 같아서요!
  3. Type Imports.tsx 파일
    who have come from flow (line 3)을 어떻게 번역해야할 지 몰라서 직역을 했는데 조금 더 나은 의견 부탁드립니다!

ref: #910

@github-actions
Copy link
Contributor

Thanks for the PR!

This section of the codebase is owned by @bumkeyy - if they write a comment saying "LGTM" then it will be merged.

Copy link
Contributor

@yeonjuan yeonjuan left a comment

Choose a reason for hiding this comment

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

고생하셨습니다:)

@@ -0,0 +1,24 @@
//// { compiler: { ts: "3.8.3" } }
// TypeScripts의 다시 export 하는 기능을 통해
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
// TypeScripts의 다시 export 하는 기능을 통해
// TypeScript의 다시 export 하는 기능을 통해


// 객체 전체를 export 하고 싶다면,
// 이전 버전의 TypeScript에서
// 조금 더 자세히 알 수 있습니다:
Copy link
Contributor

Choose a reason for hiding this comment

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

When you wanted to export the full object, that becomes a little more verbose in previous versions of TypeScript:

원문의 "becomes a little more verbose" 는 "좀 더 장황해 진다" 는 의미로 해석되어야 할 것 같습니다 :)

Comment on lines 19 to 21
// TypeScript 3.8버전에서는,
// JavaScript 사양에서 더 많은 export 문의 양식을 제공하는데,
// 한 줄에 모듈을 다시 export 할 수 있도록 합니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
// TypeScript 3.8버전에서는,
// JavaScript 사양에서 더 많은 export 문의 양식을 제공하는데,
// 한 줄에 모듈을 다시 export 할 수 있도록 합니다.
// TypeScript 3.8 은,
// JavaScript 사양의 export 보다 더 많은 양식을 제공하여,
// 한 줄에 모듈을 다시 export 할 수 있도록 합니다.

조금 자연스럽게 바꾸어봤습니다 :)

Comment on lines 2 to 3
// 3.8에선 타입 import를 위해 새로운 구문들을 추가했습니다.
// 이는 흐름에서 온 사용자와 유사합니다.
Copy link
Contributor

@yeonjuan yeonjuan Sep 15, 2020

Choose a reason for hiding this comment

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

Suggested change
// 3.8에선 타입 import를 위해 새로운 구문들을 추가했습니다.
// 이는 흐름에서 온 사용자와 유사합니다.
// 3.8에선 타입 import를 위해 새로운 구문들을 추가했습니다.
// 이는 flow 사용자에게는 익숙할 것입니다.

여기서 flow 는 flow - 타입언어 를 의미하는 것 같습니다 :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아하! 그 flow를 뜻하는거였군요 ㅎㅎ
피드백 고맙습니다. 수정해서 반영했습니다~ :)

// 값이 재할당 되었을 때 const 변수를 let으로
// 빠르게 변환하는 것입니다.

// 아래 오류에 하이라이트 표시를 하고
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
// 아래 오류에 하이라이트 표시를 하고
// 아래 오류에 하이라이트 표시를 확인하고

주체가 조금 다른 것 같아요 :)

@bumkeyy
Copy link
Contributor

bumkeyy commented Sep 18, 2020

LGTM

@github-actions github-actions bot merged commit 59b25a8 into microsoft:v2 Sep 18, 2020
@github-actions
Copy link
Contributor

Merging because @bumkeyy is a code-owner of all the changes - thanks!

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