Skip to content

Translation 3 files to Ko #1040

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

Merged
merged 6 commits into from
Sep 12, 2020
Merged

Translation 3 files to Ko #1040

merged 6 commits into from
Sep 12, 2020

Conversation

seohyun0120
Copy link
Contributor

@ghost
Copy link

ghost commented Sep 4, 2020

CLA assistant check
All CLA requirements met.

- npm 사용하기(Node.js 패키지 매니저)
- Visual Studio TypeScript 플러그인 설치하기

Visual Studio 2017과 Visual Studio 2015 Update 3에는 TypeScript가 기본적으로 포함되어있습니다.
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
Visual Studio 2017과 Visual Studio 2015 Update 3에는 TypeScript가 기본적으로 포함되어있습니다.
Visual Studio 2017과 Visual Studio 2015 Update 3에는 TypeScript가 기본적으로 포함되어 있습니다.

Visual Studio 2017과 Visual Studio 2015 Update 3에는 TypeScript가 기본적으로 포함되어있습니다.
만약 Visual Studio로 TypeScript를 다운받지 않았다면, [여기서 다운로드받으세요](/download).

npm 사용자들:
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
npm 사용자들:
npm 사용자의 경우:

- Visual Studio TypeScript 플러그인 설치하기

Visual Studio 2017과 Visual Studio 2015 Update 3에는 TypeScript가 기본적으로 포함되어있습니다.
만약 Visual Studio로 TypeScript를 다운받지 않았다면, [여기서 다운로드받으세요](/download).
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
만약 Visual Studio로 TypeScript를 다운받지 않았다면, [여기서 다운로드받으세요](/download).
만약 Visual Studio로 TypeScript를 설치하지 않았다면, [여기서 다운로드하세요](/download).


#### 타입 검사와 d.ts 파일 생성 (Type Checking and d.ts file generation)

Babel 사용의 단점은 TS를 JS로 전환하는 동안 타입 검사를 할 수 없다는 점입니다. 즉, 에디터에서 타입 오류를 잡지 못하고 상용 코드에 포함될 수도 있단 뜻입니다.
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
Babel 사용의 단점은 TS를 JS로 전환하는 동안 타입 검사를 할 수 없다는 점입니다. 즉, 에디터에서 타입 오류를 잡지 못하고 상용 코드에 포함될 수도 있단 뜻입니다.
Babel 사용의 단점은 TS를 JS로 전환하는 동안 타입 검사를 할 수 없다는 점입니다. 즉, 에디터에서 잡지 못한 타입 오류가 프로덕션 코드에 포함될 수도 있다는 뜻입니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

훨씬 더 깔끔해진 것 같아요 :)

Copy link
Contributor

@dvlprsh dvlprsh left a comment

Choose a reason for hiding this comment

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

LGTM 👍

- 빌드 출력 결과와 소스 입력 파일이 거의 비슷한가요? `tsc`를 사용하세요.
- 여러 잠재적인 결과물을 내는 빌드 파이프라인이 필요하신가요? `babel`로 트랜스파일링을 하고, `tsc`로 타입을 검사하세요.

## 트랜스파일링은 Babel, 타입은 `tsc` (Babel for transpiling, `tsc` for types)
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
## 트랜스파일링은 Babel, 타입은 `tsc` (Babel for transpiling, `tsc` for types)
## 트랜스파일링은 Babel, 타입은 `tsc`

Copy link
Contributor

Choose a reason for hiding this comment

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

현재 레포에서는 다른 국가들의 번역 규칙에 따라 원문을 삽입하지 않기로 했습니다 :)


Babel은 TypeScript를 지원하기 때문에, 기존 빌드 파이프라인으로 작업할 수 있고 Babel이 코드 타입을 검사하지 않기 때문에 JS 출력 시간이 더 빨라질 수 있습니다.

#### 타입 검사와 d.ts 파일 생성 (Type Checking and d.ts file generation)
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
#### 타입 검사와 d.ts 파일 생성 (Type Checking and d.ts file generation)
#### 타입 검사와 d.ts 파일 생성

@bumkeyy
Copy link
Contributor

bumkeyy commented Sep 10, 2020

고생하셨습니다 👍

@bumkeyy
Copy link
Contributor

bumkeyy commented Sep 12, 2020

LGTM

@github-actions github-actions bot merged commit b1ea14d into microsoft:v2 Sep 12, 2020
@github-actions
Copy link
Contributor

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

@seohyun0120 seohyun0120 deleted the translation-ko branch September 13, 2020 15:01
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.

4 participants