New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🌐 Add Turkish translation for docs/tr/docs/async.md
#5191
🌐 Add Turkish translation for docs/tr/docs/async.md
#5191
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5191 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 540 532 -8
Lines 13969 13684 -285
==========================================
- Hits 13969 13684 -285 ☔ View full report in Codecov by Sentry. |
📝 Docs preview for commit 0d74123 at: https://62dd532051a2e062611b7fcd--fastapi.netlify.app |
docs/async.md
docs/async.md
docs/tr/docs/async.md
Co-authored-by: Hakan Celik <hakancelik96@outlook.com>
📝 Docs preview for commit 486cd3a at: https://636d1c234d54b34385c0848c--fastapi.netlify.app |
📝 Docs preview for commit 8d27bef at: https://639ce180a12b8e030c9de4ea--fastapi.netlify.app |
📝 Docs preview for commit 102f276 at: https://6438b62f6785b16334cdc95f--fastapi.netlify.app |
Overall, I think it's good, there are just some small errors that cause a shift in meaning. After updating the PR, I'll take another look at all of them, thank you! |
Co-authored-by: Hakan Celik <hakancelik96@outlook.com>
📝 Docs preview for commit d532dcf at: https://647ba939a098e03b81bfccc9--fastapi.netlify.app |
📝 Docs preview for commit b7b3e76 at: https://649a2fbb219954200e358294--fastapi.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 simple change requests. Happy to approve after they are addressed.
docs/tr/docs/async.md
Outdated
|
||
Ancak, Makine Öğrenimi sistemlerindekile gibi **CPU'ya bağlı** iş yükleri için paralellik ve çoklu işlemenin (birden çok işlemin paralel olarak çalışması) avantajlarından da yararlanabilirsiniz. | ||
|
||
That, plus the simple fact that Python is the main language for **Data Science**, Machine Learning and especially Deep Learning, make FastAPI a very good match for Data Science / Machine Learning web APIs and applications (among many others). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fazlalik kalmis gibi, silinecek galiba?
docs/tr/docs/async.md
Outdated
return burgers | ||
``` | ||
|
||
...instead of `def`: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"def
yerine:" ile degistirilmesi uygun olur.
📝 Docs preview for commit 8715aca at: https://dc139c84.fastapitiangolo.pages.dev |
Thanks @BilalAlpaslan 🤓 And thanks for the reviews @cun3yt and @hakancelikdev ✨ |
docs/tr/docs/async.md
docs/tr/docs/async.md
Turkish translate for async.md
Main issue: #1892