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 Functions and noImplicitAny to ko #1174

Merged
merged 5 commits into from
Oct 10, 2020

Conversation

ginger-kang
Copy link
Contributor

@ginger-kang ginger-kang commented Oct 1, 2020

en :

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

noImplicitAny.md에서 주석 처리된 no error?를 어떻게 번역할지 고민하다가 생각이 안나 원문 그대로 두었습니다. 의견 부탁드립니다 ( _ _ )
(스프레드 시트 수정 권한이 없는 것 같아서 진행 상황 업데이트를 못했습니다. 혹시 확인 부탁드려도 될까요?)

ref #910

@ghost
Copy link

ghost commented Oct 1, 2020

CLA assistant check
All CLA requirements met.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 1, 2020

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.

//// { order: 2, compiler: { noImplicitAny: false } }

// JavaScript에서 함수를 선언하는 방식에는
// 여러 가지가 있습니다. 두 개의 number를 더하는
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
// 여러 가지가 있습니다. 두 개의 number를 더하는
// 여러 가지가 있습니다. 두 숫자를 더하는

return x + y;
};

// 우리는 마지막 방식에 초점을 맞추겠습니다. 그러나 모든 것이
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.

[제안]

// 우리는 마지막 방식에 초점을 맞추겠습니다. 그러나 모든 것이
// 세 가지 방식에 똑같이 적용됩니다.

// 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는 함수 정의에

// 추가 구문을 제공하고 해당 함수에서
// 예상되는 타입에 대한 힌트를 제공합니다.
//
// 다음은 add 함수의 가장 개방적인 버전으로,
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
// 다음은 add 함수의 가장 개방적인 버전으로,
// 다음은 add 함수의 가장 열려있는 버전으로,

[제안]

//
// 다음은 add 함수의 가장 개방적인 버전으로,
// add 함수는 any 타입의 두 개의 입력을 받습니다: 이것은
// 당신이 만든 strings, numbers 또는 objects가 될 수 있습니다.
Copy link
Contributor

@bumkeyy bumkeyy Oct 5, 2020

Choose a reason for hiding this comment

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

Suggested change
// 당신이 만든 strings, numbers 또는 objects가 될 수 있습니다.
// 문자열, 숫자 또는 객체가 될 수 있습니다.

};
add1("Hello", 23);

// JavaScript에서 이것은 허용이 됩니다(예시와 같이 strings도
Copy link
Contributor

@bumkeyy bumkeyy Oct 5, 2020

Choose a reason for hiding this comment

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

Suggested change
// JavaScript에서 이것은 허용이 됩니다(예시와 같이 strings도
// JavaScript에서는 타당하지만 (예시와 같이 문자열도

add1("Hello", 23);

// JavaScript에서 이것은 허용이 됩니다(예시와 같이 strings도
// 더해질 수 있습니다) 그러나 우리가 알고있는 number를 위한
Copy link
Contributor

@bumkeyy bumkeyy Oct 5, 2020

Choose a reason for hiding this comment

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

Suggested change
// 더해질 수 있습니다) 그러나 우리가 알고있는 number를 위한
// 더해질 수 있습니다) 우리가 의도한 숫자를 더하는


// JavaScript에서 이것은 허용이 됩니다(예시와 같이 strings도
// 더해질 수 있습니다) 그러나 우리가 알고있는 number를 위한
// 함수와는 맞지 않으므로, 우리는 x와 y를 numbers만을
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
// 함수와는 맞지 않으므로, 우리는 x와 y를 numbers만을
// 함수와는 맞지 않으므로, x와 y를 숫자만

Copy link
Contributor

Choose a reason for hiding this comment

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

조금 자연스럽게 바꿔봤습니다.

타입이 아닌 numbers, objects, strings는 번역을 하는게 나은 것 같아요.
그리고 우리, 너 와같은 주어는 생략하는 편이 읽을 때 덜 어색할 수 있어요 :)

add2(16, 23);
add2("Hello", 23);

// 좋습니다. 이제 number이외의 값이 전달되면 에러가
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
// 좋습니다. 이제 number이외의 값이 전달되면 에러가
// 좋습니다. 이제 숫자 이외의 값이 전달되면 에러가

// 위해 명시적인 반환 유형 구문을 사용하는 경우가
// 있습니다. 테스트 기반 개발에서 실패한 테스트로
// 시작하는 것이 권장되는 방식과 비슷하지만, 이 경우에는
// 대신 실패한 함수 모양이 있습니다.
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
// 대신 실패한 함수 모양이 있습니다.
// 대신 실패한 함수 형태가 있습니다.


// 이것은 add2의 매우 명시적인 버전입니다
// - 시작 하기 전에 자신에게 작업공간을 제공하기
// 위해 명시적인 반환 유형 구문을 사용하는 경우가
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
// 위해 명시적인 반환 유형 구문을 사용하는 경우가
// 위해 명시적인 반환 타입 구문을 사용하는 경우가

oneline: "Avoid introducing anys inside your codebase when a type could be specified"
---

타입 표기가 존재하지 않는 경우에 대해서, TypeScript는 타입을 추론할 수 없을 때 해당 변수의 타입을 `any`로 대체합니다.
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는 타입을 추론할 수 없을 때 해당 변수의 타입을 `any`로 대체합니다.
타입 표기가 존재하지 않을 때, TypeScript는 타입을 추론할 수 없을 때 해당 변수의 타입을 `any`로 대체합니다.

fn(42);
```

그러나 `noImplicitAny`를 켜면 TypeScript는 `any`를 추론 할 때마다 오류를 발생시킵니다:
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
그러나 `noImplicitAny`켜면 TypeScript는 `any`를 추론 할 때마다 오류를 발생시킵니다:
그러나 `noImplicitAny`활성화하면 TypeScript는 `any`를 추론 할 때마다 오류를 발생시킵니다:

@bumkeyy
Copy link
Contributor

bumkeyy commented Oct 5, 2020

고생하셨습니다 👍

noImplicitAny.md에서 주석 처리된 no error?를 어떻게 번역할지 고민하다가 생각이 안나 원문 그대로 두었습니다. 의견 부탁드립니다 ( _ _ )

오류가 아닌가요? 는 어떨까요?

@ginger-kang
Copy link
Contributor Author

@bumkeyy 리뷰 감사합니다 :)

오류가 아닌가요? 는 어떨까요?

좋은 것 같습니다!

oneline: "Avoid introducing anys inside your codebase when a type could be specified"
---

타입 표기가 존재하지 않을 때, TypeScript는 타입을 추론할 수 없을 때 해당 변수의 타입을 `any`로 대체합니다.
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는 타입을 추론할 수 없을 때 해당 변수의 타입을 `any`로 대체합니다.
타입이 표기되어 있지 않을 때, TypeScript는 타입을 추론할 수 없을 때 해당 변수의 타입을 `any`로 대체합니다.

[제안]

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@bumkeyy
'타입이 표기되어 있지 않아, 타입 추론을 할 수 없을 때 TypeScript는 해당 변수의 타입을 any로 대체합니다.' 는 어떨까요?

Copy link
Contributor

Choose a reason for hiding this comment

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

훨씬 자연스럽네요 👍

@bumkeyy
Copy link
Contributor

bumkeyy commented Oct 10, 2020

LGTM

@github-actions github-actions bot merged commit 05272bd into microsoft:v2 Oct 10, 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