Skip to content
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 Russian translation for docs/ru/docs/features.md #5315

Merged
merged 5 commits into from Sep 4, 2022
Merged

🌐 Add Russian translation for docs/ru/docs/features.md #5315

merged 5 commits into from Sep 4, 2022

Conversation

Xewus
Copy link
Contributor

@Xewus Xewus commented Aug 29, 2022

No description provided.

@codecov
Copy link

codecov bot commented Aug 29, 2022

Codecov Report

Merging #5315 (c38278e) into master (0bb8920) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master     #5315   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          535       539    +4     
  Lines        13825     13902   +77     
=========================================
+ Hits         13825     13902   +77     
Impacted Files Coverage Δ
fastapi/utils.py 100.00% <0.00%> (ø)
fastapi/routing.py 100.00% <0.00%> (ø)
tests/test_ws_router.py 100.00% <0.00%> (ø)
fastapi/openapi/utils.py 100.00% <0.00%> (ø)
...tutorial/test_custom_response/test_tutorial009c.py 100.00% <0.00%> (ø)
tests/test_enforce_once_required_parameter.py 100.00% <0.00%> (ø)
tests/test_orjson_response_class.py 100.00% <0.00%> (ø)
docs_src/custom_response/tutorial009c.py 100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Xewus Xewus changed the title ru translate for docs/features.md Add Russian translation for docs/features.md Aug 29, 2022
@github-actions
Copy link
Contributor

📝 Docs preview for commit cefba3d at: https://630ca90efbefa91f61c710c1--fastapi.netlify.app

docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
@github-actions
Copy link
Contributor

📝 Docs preview for commit 715534a at: https://630d07d67352f511684c1bf0--fastapi.netlify.app

docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
docs/ru/docs/features.md Outdated Show resolved Hide resolved
Copy link
Contributor

@Winand Winand left a comment

Choose a reason for hiding this comment

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

1 ошибка и пара замечаний. В целом всё хорошо.

@github-actions
Copy link
Contributor

📝 Docs preview for commit e0a7cf2 at: https://630dda3cf8bb5326256f97cf--fastapi.netlify.app

@impocode
Copy link
Contributor

impocode commented Aug 30, 2022

#5312
Я тут недавно тоже пушил этот файл. Посмотрите, возможно там вы найдёте то, что можно взять на заметку.

Copy link
Contributor Author

@Xewus Xewus left a comment

Choose a reason for hiding this comment

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

,,,

@Xewus
Copy link
Contributor Author

Xewus commented Aug 30, 2022

#5312 Я тут недавно тоже пушил этот файл. Посмотрите, возможно там вы найдёте то, что можно взять на заметку.

Сорри. У меня с весны валяется несколько переводов, в том числе и этот. Но низкая активность русскоязычного сообщества в переводах оказывает влияние.

@KaterinaSolovyeva
Copy link

KaterinaSolovyeva commented Aug 30, 2022

Всё отлично. Если придираться, то в одном месте только я бы упростила:
вместо:
"Разработан, придерживаясь этих стандартов, после тщательного их изучения. Эти стандарты изначально включены во фреймфорк, а не являются дополнительной надстройкой."
"Разработан, придерживаясь этих стандартов, после тщательного их изучения. Без дополнительной надстройки."

@Xewus
Copy link
Contributor Author

Xewus commented Aug 30, 2022

Всё отлично. Если придираться, то в одном месте только я бы упростила: вместо: "Разработан, придерживаясь этих стандартов, после тщательного их изучения. Эти стандарты изначально включены во фреймфорк, а не являются дополнительной надстройкой." "Разработан, придерживаясь этих стандартов, после тщательного их изучения. Без дополнительной надстройки."

Давайте будем считать это более широким уточнением, к тому же взятым из оригинала.

@Xewus
Copy link
Contributor Author

Xewus commented Aug 30, 2022

Excuse me, Mr. @tiangolo, but your actions are needed here.

@impocode
Copy link
Contributor

impocode commented Sep 3, 2022

@Xewus

Привет! На сколько я понял название коммита по переводу должно в таком стиле оформляться:

🌐 Add Russian translation for docs/ru/docs/features.md

Возможно, стоит поменять) Может тогда быстрей сольёт😅

@Xewus Xewus changed the title Add Russian translation for docs/features.md Add Russian translation for docs/ru/docs/features.md Sep 3, 2022
@Xewus
Copy link
Contributor Author

Xewus commented Sep 3, 2022

Возможно, стоит поменять) Может тогда быстрей сольётsweat_smile

Спасибо. Попробуем.

@Xewus Xewus changed the title Add Russian translation for docs/ru/docs/features.md 🌐 Add Russian translation for docs/ru/docs/features.md Sep 3, 2022
docs/ru/mkdocs.yml Outdated Show resolved Hide resolved
@tiangolo tiangolo changed the title 🌐 Add Russian translation for docs/ru/docs/features.md 🌐 Add Russian translation for docs/ru/docs/features.md Sep 4, 2022
docs/ru/mkdocs.yml Outdated Show resolved Hide resolved
@tiangolo tiangolo added lang-all Translations lang-ru Russian translations approved-2 labels Sep 4, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Sep 4, 2022

📝 Docs preview for commit c38278e at: https://6314b558d8aeff751a33fda1--fastapi.netlify.app

@tiangolo
Copy link
Owner

tiangolo commented Sep 4, 2022

Awesome, thank you @Xewus! 🍰

And thanks a lot for all the reviews and help @Winand, @mbroton, @KaterinaSolovyeva, @Winand ☕ 🍪

@tiangolo tiangolo merged commit 81967f8 into tiangolo:master Sep 4, 2022
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-ru Russian translations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants