Skip to content

🌐 Add Korean translation for docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md#13257

Merged
alejsdev merged 18 commits into
fastapi:masterfrom
11kkw:korean-translation-dependencies-with-yield
Feb 9, 2025
Merged

🌐 Add Korean translation for docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md#13257
alejsdev merged 18 commits into
fastapi:masterfrom
11kkw:korean-translation-dependencies-with-yield

Conversation

@11kkw

@11kkw 11kkw commented Jan 25, 2025

Copy link
Copy Markdown
Contributor

내용

이 PR은 docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md 파일에 대한 한국어 번역을 추가합니다. FastAPI 프로젝트에 처음으로 기여하는 PR이며, 공식 문서를 기반으로 번역을 완료했습니다.

참고 사항

  • 기존 PR(🌐 Add Korean translation for docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md #4087)이 같은 파일을 번역하는 작업으로 보입니다. 그러나 해당 PR은 2022년 7월 4일에 닫혔고, 이후 병합되지 않았습니다.
  • 이전 PR이 왜 병합되지 않았는지 명확하지 않아 새로운 작업을 진행하게 되었습니다. 혹시 병합되지 않은 이유가 있다면 알려주시면 감사하겠습니다.
  • 이번 번역 작업은 최신 FastAPI 문서를 기반으로 진행되었습니다.

번역 기준

  • 번역 과정에서 다음 용어를 사용했습니다:
    • Dependency: 의존성
    • Path Operation: 경로 작업
    • HTTPException: HTTP 예외
  • 문서의 원본 포맷(Markdown 구조 및 코드 블록)을 그대로 유지했습니다.

작업 이력

  • 관련 커밋: 🌐 Add Korean translation for docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md

처음 기여에 대한 메시지

  • 이번 PR은 저의 첫 번째 오픈소스 기여입니다. 혹시나 포맷, 프로세스, 또는 번역 스타일에 부족한 부분이 있다면 피드백을 주시면 감사하겠습니다. 기여자로서 성장할 수 있도록 배울 준비가 되어 있습니다.

검토 요청


PR에 대한 피드백과 조언을 적극적으로 반영하겠습니다. 새로운 기여자를 위한 도움을 주셔서 감사합니다!

@github-actions github-actions Bot added the lang-all Translations label Jan 25, 2025
@github-actions

Copy link
Copy Markdown
Contributor

@11kkw 11kkw changed the title 🌐 Add Korean translation for docs/ko/docs/tutorial/dependencies/depen… 🌐 Add Korean translation for docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md Jan 26, 2025
@github-actions

Copy link
Copy Markdown
Contributor

@11kkw 11kkw changed the title 🌐 Add Korean translation for docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md 🌐 Add Korean translation for docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md Jan 26, 2025
@11kkw 11kkw changed the title 🌐 Add Korean translation for docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md 🌐 Add Korean translation for docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md Jan 26, 2025
@11kkw 11kkw changed the title 🌐 Add Korean translation for docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md 🌐 Add Korean translation for docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md Jan 26, 2025
@svlandeg svlandeg added awaiting-review lang-ko Korean translations labels Jan 27, 2025
Comment thread docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md Outdated
Comment thread docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md Outdated
Comment thread docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md Outdated

/// note | 기술 세부사항

파이썬의 <a href=“https://docs.python.org/3/library/contextlib.html” class=“external-link” target=“_blank”>Context Managers</a> 덕분에 이 기능이 작동합니다.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

<a href="https://docs.python.org/3/library/contextlib.html" class="external-link" target="_blank"> 링크가 원문과 동일한데 아래와 같이 링크가 깨지는군요
GitHub Actions에서 설정 문제가 있나보네요

Screenshot 2025-02-01 at 6 36 51 PM

Comment thread docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md Outdated
Comment thread docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md Outdated
Comment thread docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md Outdated
Comment thread docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md Outdated
Comment thread docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md Outdated
Comment thread docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md Outdated
11kkw and others added 11 commits February 1, 2025 19:34
Co-authored-by: Hyogeun Oh (오효근) <ohg3417@gmail.com>
Co-authored-by: Hyogeun Oh (오효근) <ohg3417@gmail.com>
Co-authored-by: Hyogeun Oh (오효근) <ohg3417@gmail.com>
Co-authored-by: Hyogeun Oh (오효근) <ohg3417@gmail.com>
Co-authored-by: Hyogeun Oh (오효근) <ohg3417@gmail.com>
Co-authored-by: Hyogeun Oh (오효근) <ohg3417@gmail.com>
Co-authored-by: Hyogeun Oh (오효근) <ohg3417@gmail.com>
Co-authored-by: Hyogeun Oh (오효근) <ohg3417@gmail.com>
Co-authored-by: Hyogeun Oh (오효근) <ohg3417@gmail.com>
Co-authored-by: Hyogeun Oh (오효근) <ohg3417@gmail.com>
Co-authored-by: Hyogeun Oh (오효근) <ohg3417@gmail.com>

예를 들어, 이 기능을 사용하면 데이터베이스 세션을 생성하고 작업이 끝난 후에 세션을 종료할 수 있습니다.

응답을 생성하기 전에는 `yield`를 포함하여 그 이전의 코드만이 실행됩니다:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

At line 38, "yield statement" is translated as "yield 문", but at line 30, it is translated simply as "yield". It would be more appropriate to consistently use "yield 문" in both cases.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@timothy-jeong Thanks for the feedback! I have applied the suggested changes. 🚀


## `yield`를 사용하는 하위 의존성

어떤 크기와 형태의 하위 의존성과 하위 의존성의 "트리"도 가질 수 있으며, 이들 모두가 `yield`를 사용할 수 있습니다.

@timothy-jeong timothy-jeong Feb 7, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

n the sentence "You can have sub-dependencies and 'trees' of sub-dependencies of any size and shape, and any or all of them can use yield," rather than explicitly translating "any" as "어떤," it would be more natural to translate it as "모든" (all).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@timothy-jeong Thanks for the feedback! I have applied the suggested changes. 🚀

@github-actions

github-actions Bot commented Feb 7, 2025

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 488e38d at: https://f9de835e.fastapitiangolo.pages.dev

Modified Pages

@github-actions

github-actions Bot commented Feb 8, 2025

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Feb 8, 2025

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Feb 8, 2025

Copy link
Copy Markdown
Contributor

@11kkw 11kkw left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I have addressed all the requested changes. Thanks for the review! 🚀

@timothy-jeong timothy-jeong left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great Work!

@github-actions github-actions Bot removed the waiting label Feb 8, 2025
@alejsdev alejsdev merged commit ad33193 into fastapi:master Feb 9, 2025
@alejsdev

alejsdev commented Feb 9, 2025

Copy link
Copy Markdown
Member

Thanks for your work! @11kkw 🏅
And thanks for your help! @Zerohertz @timothy-jeong 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved-2 lang-all Translations lang-ko Korean translations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants