Skip to content

🌐 Add Korean translation for docs/ko/docs/benchmarks.md and docs/ko/docs/tutorial/testing.md#9496

Closed
ElliottLarsen wants to merge 8 commits intofastapi:masterfrom
ElliottLarsen:master
Closed

🌐 Add Korean translation for docs/ko/docs/benchmarks.md and docs/ko/docs/tutorial/testing.md#9496
ElliottLarsen wants to merge 8 commits intofastapi:masterfrom
ElliottLarsen:master

Conversation

@ElliottLarsen
Copy link
Contributor

@ElliottLarsen ElliottLarsen commented May 8, 2023

Add Korean translation for docs/ko/docs/tutorial/testing.md
Related to Korean translations #2017

@github-actions
Copy link
Contributor

github-actions bot commented May 8, 2023

@tiangolo tiangolo changed the title Add Korean translation for docs/ko/docs/tutorial/testing.md 🌐 Add Korean translation for docs/ko/docs/tutorial/testing.md May 8, 2023
@tiangolo tiangolo added lang-all Translations awaiting-review lang-ko Korean translations labels May 8, 2023
@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

Merge from FastAPI Master
@github-actions
Copy link
Contributor

@tiangolo
Copy link
Member

@tiangolo tiangolo changed the title 🌐 Add Korean translation for docs/ko/docs/tutorial/testing.md 🌐 Add Korean translation for docs/ko/docs/benchmarks.md and docs/ko/docs/tutorial/testing.md Jun 27, 2023
@joonas-yoon
Copy link
Contributor

Thanks for your work. I'm not sure about two document changes at one PR. I think it would be seperated.

@hard-coders
Copy link
Contributor

You have to separate it. Close the PR and open it again 😿

@tiangolo
Copy link
Member

It would be better to do this in 2 PRs, that way others would be able to review and approve one file and I could merge it. We could do that and continue gradually. 🤓

For now, I'll close this one. But feel free to create 2 new PRs (or anyone else that sees this, feel free to copy the contents and start a new PR with it, or start over, as the original might have changed a bit). ☕

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

Labels

lang-all Translations lang-ko Korean translations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants