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

🌐 Fix Korean translation fordocs/ko/docs/index.md#4195

Merged
tiangolo merged 2 commits intofastapi:masterfrom
kty4119:docs-ko-docs-index.md
Jan 16, 2022
Merged

🌐 Fix Korean translation fordocs/ko/docs/index.md#4195
tiangolo merged 2 commits intofastapi:masterfrom
kty4119:docs-ko-docs-index.md

Conversation

@kty4119
Copy link
Contributor

@kty4119kty4119 commentedNov 20, 2021•
edited
Loading

Thank you always for your hard work.
I found something good to fix in this file.

  • Change the expression that doesn't match.
  • changed it according to the guidelines.

@hard-coders
Please check if this modification is appropriate and leave a comment.
Thank you for your hard work.

This PR complements docs/ko/docs/index.md
related :#2017

Copy link
Contributor

@DevDaeDevDae left a comment

Choose a reason for hiding this comment

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

Thank you for your works.
Please check my PR too! 😄

Copy link
Contributor

@hard-codershard-coders left a comment

Choose a reason for hiding this comment

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

This PR is following the rules better. Thank you for your effort.

@tiangolotiangolo changed the title🌐 Fix Korean translation for docs/ko/docs/index.md🌐 Fix Korean translation fordocs/ko/docs/index.mdDec 8, 2021
@tiangolotiangolo added awaiting-review lang-allTranslations lang-koKorean translations labelsDec 8, 2021
@github-actionsgithub-actionsbot mentioned this pull requestDec 8, 2021
@NinaHwang
Copy link
Contributor

LGTM 🚀

@tiangolo
Copy link
Member

Awesome, thank you@kty4119! 🍰

And thanks for the reviews@hard-coders,@NinaHwang,@DevDae. 🙇 ☕

@tiangolotiangoloenabled auto-merge (squash)January 16, 2022 19:39
@codecov
Copy link

codecovbot commentedJan 16, 2022•
edited
Loading

Codecov Report

Merging#4195 (9daf262) intomaster (0a61a6c) willnot change coverage.
The diff coverage isn/a.

Impacted file tree graph

@@             Coverage Diff             @@##            master     #4195     +/-   ##===========================================  Coverage   100.00%   100.00%             ===========================================  Files          409       504     +95       Lines        10264     12707   +2443     ===========================================+ Hits         10264     12707   +2443
Impacted FilesCoverage Δ
fastapi/utils.py100.00% <0.00%> (ø)
fastapi/applications.py100.00% <0.00%> (ø)
fastapi/openapi/docs.py100.00% <0.00%> (ø)
...test_tutorial/test_dataclasses/test_tutorial002.py100.00% <0.00%> (ø)
..._tutorial/test_sql_databases/test_sql_databases.py100.00% <0.00%> (ø)
...orial/test_sql_databases/test_testing_databases.py100.00% <0.00%> (ø)
docs_src/app_testing/main_b.py
docs_src/app_testing/test_main_b.py
...utorial/test_body_updates/test_tutorial001_py39.py100.00% <0.00%> (ø)
docs_src/extra_models/tutorial005_py39.py100.00% <0.00%> (ø)
... and95 more

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last update0a61a6c...9daf262. Read thecomment docs.

@tiangolotiangolo merged commit26e9411 intofastapi:masterJan 16, 2022
JeanArhancet pushed a commit to JeanArhancet/fastapi that referenced this pull requestAug 20, 2022
Co-authored-by: Sebastián Ramírez <tiangolo@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

3 more reviewers

@hard-codershard-codershard-coders approved these changes

@NinaHwangNinaHwangNinaHwang approved these changes

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

@kty4119@NinaHwang@tiangolo@hard-coders@DevDae
[8]ページ先頭
©2009-2026 Movatter.jp