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

Translation 2 files to Ko #1081

Merged
merged 1 commit into from
Sep 27, 2020
Merged

Conversation

zigsong
Copy link
Contributor

@zigsong zigsong commented Sep 13, 2020

en:

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

ref: #910

@ghost
Copy link

ghost commented Sep 13, 2020

CLA assistant check
All CLA requirements met.

@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.

@zigsong
Copy link
Contributor Author

zigsong commented Sep 17, 2020

파일이 하나 안 들어가있어서 이제 CI가 다 돌았네요 ㅠㅠ
리뷰 부탁드리겠습니다! @yeonjuan @guyeol @dvlprsh

@@ -0,0 +1,83 @@
// 객체의 형태를 선언하는 두 가지 주요 도구가 있습니다:
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
// 객체의 형태를 선언하는 두 가지 주요 도구가 있습니다:
// 객체 형태를 선언하는 주요 도구는 두 가지가 있습니다.

[제안]

// 인터페이스와 타입 별칭입니다.
//
// 이 둘은 무척 비슷하며,
// 대부분의 경우 동일하게 작동합니다.
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
// 대부분의 경우 동일하게 작동합니다.
// 대부분 동일하게 작동합니다.

const bird2: BirdInterface = { wings: 2 };

// 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
// 이들을 교차하여 사용하는 것도 가능합니다.
// 교차하여 사용하는 것도 가능합니다.

[제안]


// 둘 다 다른 인터페이스와 타입으로의 확장이 가능합니다.
// 타입 별칭은 교차 타입을 통해 이를 수행하는 반면,
// 인터페이스는 키워드를 가집니다.
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
// 인터페이스는 키워드를 가집니다.
// 인터페이스는 키워드를 사용합니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

extend 키워드를 사용해서 확장하는 의미인 것 같아요 :)

let owl: Owl = { wings: 2, nocturnal: true };
let chicken: Chicken = { wings: 2, colourful: false, flies: false };

// 그렇긴 하지만, 타입 별칭 대신 인터페이스를 쓰는 것을 추천합니다.
Copy link
Contributor

@bumkeyy bumkeyy Sep 18, 2020

Choose a reason for hiding this comment

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

Suggested change
// 그렇긴 하지만, 타입 별칭 대신 인터페이스를 쓰는 것을 추천합니다.
// 그래도 타입 별칭 보다 인터페이스를 쓰는 것을 추천합니다.

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.

'별칭보다'로 (띄어쓰기 없이) 수정하겠습니다! :D 좋은 의견인 것 같아요

// 그렇긴 하지만, 타입 별칭 대신 인터페이스를 쓰는 것을 추천합니다.
// 분명히 더 나은 에러 메시지를 받을 수 있기 때문입니다.
// 뒤따르는 에러에 커서를 가져가면,
// Chicken과 같은 인터페이스를 사용했을 때
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
// Chicken과 같은 인터페이스를 사용했을 때
// Chicken 인터페이스를 사용했을 때

[제안]

Copy link
Contributor Author

Choose a reason for hiding this comment

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

본문이 like Chicken인데, Chicken 인터페이스에만 해당되는 것이 아니라 같은 형태의 인터페이스에도 해당된다는 의미로 Chicken과 같은으로 해석해야 하지 않나요??

Copy link
Contributor

Choose a reason for hiding this comment

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

다시보니 원래 번역이 더 맞는거 같네요!

// 분명히 더 나은 에러 메시지를 받을 수 있기 때문입니다.
// 뒤따르는 에러에 커서를 가져가면,
// Chicken과 같은 인터페이스를 사용했을 때
// 어떻게 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
// 어떻게 TypeScript가 더 간결하고 정확한 메시지를 제공하는지 알 수 있습니다.
// TypeScript가 더 간결하고 정확한 메시지를 어떻게 제공하는지 알 수 있습니다.

[제안]

// 그것들을 인터페이스로 만드는 것이 더 좋습니다.

// 타입과 인터페이스에 대한 모든 엣지 케이스를 알고 싶다면,
// 이 stack overflow thread가
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
// stack overflow thread가
// 아래 stack overflow thread가

const okNumberIndex = StatusCodes["OK"];
const stringBadRequest = StatusCodes[400];

// 열거형은 여러 타입일 수 있지만, 문자열 타입이 일반적입니다.
Copy link
Contributor

@bumkeyy bumkeyy Sep 18, 2020

Choose a reason for hiding this comment

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

Suggested change
// 열거형은 여러 타입일 수 있지만, 문자열 타입이 일반적입니다.
// 열거형은 여러 타입일 수 있지만, 일반적으로 string 타입입니다.

[제안]

}
};

// 트랜스파일된 JavaScript를 보면,
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
// 트랜스파일된 JavaScript를 보면,
// 트랜스파일된 JavaScript 코드를 보면,

@bumkeyy
Copy link
Contributor

bumkeyy commented Sep 18, 2020

@zigsong @yeonjuan @guyeol @dvlprsh

타입을 한글로 번역하는 게 나을까요?

string => 문자열
number => 숫자

전 키워드라고 생각되서 원문 그대로 가는게 더 이해가 잘 되는것 같아요.
any, unknown 과 같은 타입은 번역하기 어려울 것 같거든요.

의견 주시면 번역 규칙에 추가할게요! :)

@zigsong
Copy link
Contributor Author

zigsong commented Sep 19, 2020

@bumkeyy any, unknown을 생각해보니 그렇네요. 저도 원문 그대로의 타입 작성이 좋은 것 같습니다!

@zigsong zigsong force-pushed the translation-ko branch 2 times, most recently from 48e6a50 to be2d1d9 Compare September 21, 2020 12:02
@bumkeyy
Copy link
Contributor

bumkeyy commented Sep 21, 2020

@zigsong
커밋 내용 중에 documentationNavigation.ts 파일은 잘못 포함된건가요?

@zigsong
Copy link
Contributor Author

zigsong commented Sep 23, 2020

@bumkeyy
master rebase하면서 commit 내역이 잘못 꼬였나봐요..! 수정했습니다

@zigsong
Copy link
Contributor Author

zigsong commented Sep 27, 2020

@bumkeyy @guyeol @dvlprsh 확인 부탁드릴게요 :)

@bumkeyy
Copy link
Contributor

bumkeyy commented Sep 27, 2020

LGTM

@github-actions github-actions bot merged commit 19223e5 into microsoft:v2 Sep 27, 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.

2 participants