optimizing > instrumentation 번역 #335

sohnjunior · 2023-06-28T13:25:22Z

기여자용

문서 개선

PR을 열기 전에 pnpm prettier-fix를 실행하여 서식 문제를 해결합니다. - 문서 기여 가이드를 읽고 문서 지침을 따르는지 확인하세요: https://github.com/Nextjs-kr/Nextjs.ko/blob/main/packages/next/README.md

Progress

pnpm prettier-fix
번역 초안 작성 (Draft translation)
공통 스타일 가이드 확인 (Check the common style guide)
모범사례 확인 (Check best practices)
용어 확인 (Check the term)
맞춤법 검사 (Spelling check)
리뷰 반영 (Resolve reviews)

closes: #121

devyuriii · 2023-06-30T00:13:29Z

ref #121

YuriKwon

고생하셨습니다!
리뷰 확인 및 반영 부탁드립니다 :)

YuriKwon · 2023-07-13T14:06:00Z

docs/02-app/01-building-your-application/05-optimizing/09-instrumentation.mdx

 ---

-If you export a function named `register` from a `instrumentation.ts` (or `.js`) file in the **root directory** of your project (or inside the `src` folder if using one), we will call that function whenever a new Next.js server instance is bootstrapped.
+프로젝트의 **루트 디렉토리**(혹은 src 디렉토리를 사용시 src 내부) 에 있는 `instrumentation.ts` (혹은 `.js`) 에서 `register` 로 명명된 함수를 export 한다면 새로운 Next.js 서버 인스턴스가 시작될 때마다 그 함수를 호출할 것입니다.


export도 번역해주세요! (ex. 내보낸다면)

그 함수 대신, 동일한 말이 반복되더라도 명확히 나타내면 더 좋을 것 같습니다!

5737dd7 29f4a67 ecfd8c1 수정했습니다.

YuriKwon · 2023-07-13T14:11:19Z

docs/02-app/01-building-your-application/05-optimizing/09-instrumentation.mdx

-> - If you use the [`pagesExtension` config option](/docs/pages/api-reference/next-config-js/pageExtensions) to add a suffix, you will also need to update the `instrumentation` filename to match.
-> - We have created a basic [with-opentelemetry](https://github.com/vercel/next.js/tree/canary/examples/with-opentelemetry) example that you can use.
+> - 이 기능은 현재 **실험적인** 단계입니다. 사용하려면 `next.config.js` 에서 `experimental.instrumentationHook = true;` 로 설정해서 명시적으로 허용해야 합니다.
+> - `instrumentation` 파일은 프로젝트 루트에 있어야 하며 `app` 이나 `pages` 디렉토리 하위에 있으면 안 됩니다. `src` 디렉토리를 사용한다면 이 파일을 `pages` 와 `app` 과 나란히 있는 `src` 하위에 위치시키세요.


Suggested change

> - `instrumentation` 파일은 프로젝트 루트에 있어야 하며 `app` 이나 `pages` 디렉토리 하위에 있으면 안 됩니다. `src` 디렉토리를 사용한다면 이 파일을 `pages` 와 `app` 과 나란히 있는 `src` 하위에 위치시키세요.

> - `instrumentation` 파일은 `app` 이나 `pages` 디렉토리 하위가 아닌 프로젝트 루트에 있어야 합니다. `src` 디렉토리를 사용한다면 이 파일을 `pages` 와 `app` 과 나란히 있는 `src` 하위에 위치시키세요.

현재 내용도 좋지만, 이렇게 바꿔보면 어떨까요??
기여 가이드 >억양 부분에 나와있듯이 부정어를 최소화하는게 좋을 것 같습니다:)

d76d543 에서 수정했습니다.

YuriKwon · 2023-07-13T14:15:50Z

docs/02-app/01-building-your-application/05-optimizing/09-instrumentation.mdx

-> - We have created a basic [with-opentelemetry](https://github.com/vercel/next.js/tree/canary/examples/with-opentelemetry) example that you can use.
+> - 이 기능은 현재 **실험적인** 단계입니다. 사용하려면 `next.config.js` 에서 `experimental.instrumentationHook = true;` 로 설정해서 명시적으로 허용해야 합니다.
+> - `instrumentation` 파일은 프로젝트 루트에 있어야 하며 `app` 이나 `pages` 디렉토리 하위에 있으면 안 됩니다. `src` 디렉토리를 사용한다면 이 파일을 `pages` 와 `app` 과 나란히 있는 `src` 하위에 위치시키세요.
+> - 접미사를 추가하기 위해 [`pagesExtension` 설정 옵션](/docs/pages/api-reference/next-config-js/pageExtensions)을 사용한다면 instrumentation 파일이름 또한 일치시키기 위해 변경이 필요합니다.


현재 문장 구조에서는 변경의 대상이 잘 드러나지 않는 것 같습니다.
파일이름을 변경(업데이트)해야한다는 내용이 잘 드러나도록 수정 부탁드립니다!

update 단어의 번역을 변경 대신 업데이트로 하면 어떨까요?
변경은 change의 의미에 가까운 것 같습니다.

1번은 7323c91 과 같이 변경해봤습니다.
2번은 b7f93c1 에서 수정했습니다.

YuriKwon · 2023-07-13T14:18:55Z

docs/02-app/01-building-your-application/05-optimizing/09-instrumentation.mdx


-When your `register` function is deployed, it will be called on each cold boot (but exactly once in each environment).
+`register` 함수가 배포되면 각각의 cold boot마다 호출됩니다. (각 환경마다 정확히 한 번씩 호출됩니다.)


괄호 앞에는 온점을 삭제해주세요!

타 파트에서 cold start를 콜드 스타트로 번역한 사례가 있습니다.
cold boot또한 콜드 부트로 해석하면 어떨까요??

a76ed1b 수정했습니다.

YuriKwon · 2023-07-13T14:33:18Z

docs/02-app/01-building-your-application/05-optimizing/09-instrumentation.mdx


-Sometimes, it may be useful to import a file in your code because of the side effects it will cause. For example, you might import a file that defines a set of global variables, but never explicitly use the imported file in your code. You would still have access to the global variables the package has declared.
+경우에 따라 특정 파일이 가지고 있는 부수 효과를 위해 해당 파일을 코드에 포함하는 것이 유용할 수 있습니다. 예를 들어 전역 변수가 정의된 파일을 명시적으로 사용하지는 않지만 가져올 수 있습니다. 이를 통해 해당 패키지가 선언한 전역 변수에 접근할 수 있습니다.


번역 검수 단톡방에서 금일 논의 후, side effects 단어에 대한 번역을 사이드 이펙트로 확정하였습니다.! 따라서 부수 효과로 번역해주신 부분들 일괄 수정 부탁드립니다!

side effect의 의미를 고려했을 때, 부수 효과를 위해 보다 때문에/피하기 위해와 같이 번역하면 더 자연스러울 것 같습니다.

import 또한 포함하는 대신 불러오는으로 수정 부탁드립니다:)

09499ac 에서 수정했습니다.

YuriKwon · 2023-07-13T14:35:13Z

docs/02-app/01-building-your-application/05-optimizing/09-instrumentation.mdx

@@ -34,7 +34,7 @@ export function register() {
 }
 ```

-However, we recommend importing files with side effects using `import` from within your `register` function instead. The following example demonstrates a basic usage of `import` in a `register` function:
+그러나 이보단 부수 효과를 가지는 파일을 불러올 때는 register 함수 내에서 `import` 하는 것을 권장합니다. 다음은 `register` 함수에서 `import` 문을 사용하는 기본 예제입니다.


이보단 은 이보다는으로 수정되거나 삭제되어도 괜찮을 것 같습니다.

수정했습니다!

YuriKwon · 2023-07-13T14:47:47Z

docs/02-app/01-building-your-application/05-optimizing/09-instrumentation.mdx


-We call `register` in all environments, so it's necessary to conditionally import any code that doesn't support both `edge` and `nodejs`. You can use the environment variable `NEXT_RUNTIME` to get the current environment. Importing an environment-specific code would look like this:
+모든 환경에서 `register`를 호출하기 때문에 `edge`와 `nodejs` 를 모두 지원하지 않는 코드는 조건부 import 문으로 불러와야 합니다. 현재 환경정보는 `NEXT_RUNTIME` 환경변수로 가져올 수 있습니다. 다음은 환경별로 코드를 불러오는 예제입니다.


조건부 import 문으로 불러오라는 내용은 없는 듯 하여, 조건부 import 해야 합니다. 정도로 수정하면 좋을 것 같습니다.

현재 환경정보는 현재의 환경으로 수정하면 어떨까요?
하단의 예제를 봤을 때 환경에 대한 정보를 전부 가져온다기 보단 환경 이름 자체만 가져오는 것 같습니다!

또한 백틱 뒤에 조사가 오는 경우 붙여써주세요!

nodejs 를 -> nodejs를

다른 문장에도 일괄 수정 부탁드립니다 :)

1, 2 는 dbd863f 에서, 백틱 뒤에 조사 오는경우 공백 제거는 9446914 에서 수정했습니다.

yoo-jimin127

LGTM!! 기여해주셔서 감사합니다 :) 👍
추가로 번역 희망하시는 챕터의 경우 이슈에 작성해주셔도 됩니다!!

sohnjunior added 10 commits June 28, 2023 21:47

docs: instrumentation 번역 초안 작성

847a6de

docs: 쌍점 제거

4e5a744

docs: 쉼표 제거

d2cc306

docs: '당신' 제거

ba667cd

docs: 단수형 사용

d5e793f

docs: if 제거

5085d96

docs: 맞춤법 교정

907cc52

docs: 문맥 교정

e7e02b6

docs: 문맥 교정

dec1b3c

docs: 문맥 교정

c50d350

sohnjunior marked this pull request as ready for review June 28, 2023 13:38

sohnjunior requested a review from ChanghyeonYoon as a code owner June 28, 2023 13:38

sohnjunior added 2 commits June 28, 2023 22:39

docs: description 수정

83d65a2

docs: title 수정

f0827a8

ChanghyeonYoon added the documentation 문서 작업 label Jun 29, 2023

devyuriii assigned sohnjunior Jun 30, 2023

devyuriii mentioned this pull request Jun 30, 2023

🚀New Next.js Docs Translation Progress (Korean) #1

Open

YuriKwon self-requested a review July 8, 2023 13:38

YuriKwon added the In Review label Jul 8, 2023

YuriKwon reviewed Jul 13, 2023

View reviewed changes

sohnjunior added 9 commits July 24, 2023 23:00

docs: export 번역

5737dd7

docs: 중복 문구 수정

29f4a67

docs: 대명사 제거

ecfd8c1

docs: 긍정문으로 변경

d76d543

docs: 업데이트로 번역 수정

b7f93c1

docs: 온점 삭제하고 콜드 부트로 번역

a76ed1b

docs: 부수 효과 -> 사이드 이펙트로 번역

09499ac

docs: 어색한 문구 수정

dbd863f

백틱 뒤에 조사 사이 공백 제거

9446914

docs: 문장구조 개선

7323c91

sohnjunior requested a review from YuriKwon July 24, 2023 14:55

docs: 오역 수정

217f93f

yoo-jimin127 approved these changes Aug 20, 2023

View reviewed changes

yoo-jimin127 merged commit 0c54bce into Nextjs-kr:main Aug 20, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

optimizing > instrumentation 번역 #335

optimizing > instrumentation 번역 #335

sohnjunior commented Jun 28, 2023 •

edited

Loading

devyuriii commented Jun 30, 2023

YuriKwon left a comment

YuriKwon Jul 13, 2023

sohnjunior Jul 24, 2023 •

edited

Loading

YuriKwon Jul 13, 2023

sohnjunior Jul 24, 2023

YuriKwon Jul 13, 2023

sohnjunior Jul 24, 2023 •

edited

Loading

YuriKwon Jul 13, 2023

sohnjunior Jul 24, 2023

YuriKwon Jul 13, 2023

sohnjunior Jul 24, 2023

YuriKwon Jul 13, 2023

sohnjunior Jul 24, 2023

YuriKwon Jul 13, 2023

YuriKwon Jul 13, 2023

sohnjunior Jul 24, 2023 •

edited

Loading

yoo-jimin127 left a comment

	> - `instrumentation` 파일은 프로젝트 루트에 있어야 하며 `app` 이나 `pages` 디렉토리 하위에 있으면 안 됩니다. `src` 디렉토리를 사용한다면 이 파일을 `pages` 와 `app` 과 나란히 있는 `src` 하위에 위치시키세요.
	> - `instrumentation` 파일은 `app` 이나 `pages` 디렉토리 하위가 아닌 프로젝트 루트에 있어야 합니다. `src` 디렉토리를 사용한다면 이 파일을 `pages` 와 `app` 과 나란히 있는 `src` 하위에 위치시키세요.


		When your `register` function is deployed, it will be called on each cold boot (but exactly once in each environment).
		`register` 함수가 배포되면 각각의 cold boot마다 호출됩니다. (각 환경마다 정확히 한 번씩 호출됩니다.)


		Sometimes, it may be useful to import a file in your code because of the side effects it will cause. For example, you might import a file that defines a set of global variables, but never explicitly use the imported file in your code. You would still have access to the global variables the package has declared.
		경우에 따라 특정 파일이 가지고 있는 부수 효과를 위해 해당 파일을 코드에 포함하는 것이 유용할 수 있습니다. 예를 들어 전역 변수가 정의된 파일을 명시적으로 사용하지는 않지만 가져올 수 있습니다. 이를 통해 해당 패키지가 선언한 전역 변수에 접근할 수 있습니다.


		We call `register` in all environments, so it's necessary to conditionally import any code that doesn't support both `edge` and `nodejs`. You can use the environment variable `NEXT_RUNTIME` to get the current environment. Importing an environment-specific code would look like this:
		모든 환경에서 `register`를 호출하기 때문에 `edge`와 `nodejs` 를 모두 지원하지 않는 코드는 조건부 import 문으로 불러와야 합니다. 현재 환경정보는 `NEXT_RUNTIME` 환경변수로 가져올 수 있습니다. 다음은 환경별로 코드를 불러오는 예제입니다.

optimizing > instrumentation 번역 #335

optimizing > instrumentation 번역 #335

Conversation

sohnjunior commented Jun 28, 2023 • edited Loading

기여자용

문서 개선

Progress

devyuriii commented Jun 30, 2023

YuriKwon left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sohnjunior Jul 24, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sohnjunior Jul 24, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sohnjunior Jul 24, 2023 • edited Loading

Choose a reason for hiding this comment

yoo-jimin127 left a comment

Choose a reason for hiding this comment

sohnjunior commented Jun 28, 2023 •

edited

Loading

sohnjunior Jul 24, 2023 •

edited

Loading

sohnjunior Jul 24, 2023 •

edited

Loading

sohnjunior Jul 24, 2023 •

edited

Loading