Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Comments

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

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

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

Conversation

@11kkw
Copy link
Contributor

내용

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

참고 사항

  • 기존 PR(🌐 Add Korean translation fordocs/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-actionsgithub-actionsbot added the lang-allTranslations labelJan 25, 2025
@github-actions
Copy link
Contributor

@11kkw11kkw 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.mdJan 26, 2025
@github-actions
Copy link
Contributor

@11kkw11kkw changed the title🌐 Add Korean translation for docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md🌐 Add Korean translation fordocs/ko/docs/tutorial/dependencies/dependencies-with-yield.mdJan 26, 2025
@11kkw11kkw changed the title🌐 Add Korean translation fordocs/ko/docs/tutorial/dependencies/dependencies-with-yield.md🌐 Add Korean translation for docs/ko/docs/tutorial/dependencies/dependencies-with-yield.mdJan 26, 2025
@11kkw11kkw changed the title🌐 Add Korean translation for docs/ko/docs/tutorial/dependencies/dependencies-with-yield.md🌐 Add Korean translation fordocs/ko/docs/tutorial/dependencies/dependencies-with-yield.mdJan 26, 2025
@svlandegsvlandeg added awaiting-review lang-koKorean translations labelsJan 27, 2025

/// note | 기술 세부사항

파이썬의 <a href=“https://docs.python.org/3/library/contextlib.html” class=“external-link” target=“_blank”>Context Managers</a> 덕분에 이 기능이 작동합니다.
Copy link
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"> 링크가 원문과 동일한데 아래와 같이 링크가 깨지는군요
GitHub Actions에서 설정 문제가 있나보네요

Screenshot 2025-02-01 at 6 36 51 PM

11kkwand others added11 commitsFebruary 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
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.

Zerohertz reacted with thumbs up emoji
Copy link
ContributorAuthor

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

timothy-jeong reacted with thumbs up emoji

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

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

@timothy-jeongtimothy-jeongFeb 7, 2025
edited
Loading

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

Zerohertz reacted with thumbs up emoji
Copy link
ContributorAuthor

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

timothy-jeong reacted with thumbs up emoji
@github-actions
Copy link
Contributor

📝 Docs preview for commit488e38d at:https://f9de835e.fastapitiangolo.pages.dev

Modified Pages

Zerohertz reacted with laugh emoji

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

Copy link
ContributorAuthor

@11kkw11kkw left a comment

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 reacted with thumbs up emoji
Copy link
Contributor

@timothy-jeongtimothy-jeong left a comment

Choose a reason for hiding this comment

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

Great Work!

11kkw reacted with thumbs up emoji
@alejsdevalejsdev merged commitad33193 intofastapi:masterFeb 9, 2025
58 checks passed
@alejsdev
Copy link
Member

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

timothy-jeong, Zerohertz, and 11kkw reacted with rocket emoji

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

Reviewers

2 more reviewers

@ZerohertzZerohertzZerohertz approved these changes

@timothy-jeongtimothy-jeongtimothy-jeong approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved-2lang-allTranslationslang-koKorean translations

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@11kkw@alejsdev@Zerohertz@timothy-jeong@svlandeg

[8]ページ先頭

©2009-2026 Movatter.jp