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/deployment/versions.md #4985

Merged
merged 5 commits into from Aug 26, 2022
Merged

🌐 Add Russian translation for docs/ru/docs/deployment/versions.md #4985

merged 5 commits into from Aug 26, 2022

Conversation

emp7yhead
Copy link
Contributor

Translate docs/ru/docs/deployment/versions.md and add deploy menu in Russian.
related: #1362

@codecov
Copy link

codecov bot commented Jun 4, 2022

Codecov Report

Merging #4985 (9ff9f92) into master (1876ebc) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master     #4985   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          532       533    +1     
  Lines        13672     13755   +83     
=========================================
+ Hits         13672     13755   +83     
Impacted Files Coverage Δ
fastapi/utils.py 100.00% <0.00%> (ø)
fastapi/routing.py 100.00% <0.00%> (ø)
fastapi/encoders.py 100.00% <0.00%> (ø)
fastapi/openapi/docs.py 100.00% <0.00%> (ø)
fastapi/openapi/utils.py 100.00% <0.00%> (ø)
fastapi/openapi/constants.py 100.00% <0.00%> (ø)
fastapi/dependencies/utils.py 100.00% <0.00%> (ø)
tests/test_dependency_cache.py 100.00% <0.00%> (ø)
tests/test_jsonable_encoder.py 100.00% <0.00%> (ø)
tests/test_validate_response.py 100.00% <0.00%> (ø)
... and 2 more

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

@github-actions
Copy link
Contributor

github-actions bot commented Jun 4, 2022

📝 Docs preview for commit 8c9ee7a at: https://629b80a66924e50eb6fb1f77--fastapi.netlify.app

@tiangolo tiangolo changed the title Add Russian translation for docs/ru/docs/deployment/versions.md 🌐 Add Russian translation for docs/ru/docs/deployment/versions.md Aug 18, 2022
@tiangolo tiangolo added lang-all Translations lang-ru Russian translations awaiting-review labels Aug 18, 2022
@github-actions github-actions bot mentioned this pull request Aug 18, 2022
@@ -0,0 +1,87 @@
# О версиях FastAPI

**FastAPI** уже используется в продакшене во многих приложениях и системах. Покрытие тестами поддерживается на уровне 100%. Однако, его разработка все еще продолжается.
Copy link
Contributor

Choose a reason for hiding this comment

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

"Однако его разработка" https://www.nkj.ru/archive/articles/23602


## Доступные версии

Вы можете увидеть доступные версии (например - проверить последнюю на данный момент) в [Release Notes](../release-notes.md){.internal-link target=_blank}.
Copy link
Contributor

Choose a reason for hiding this comment

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

(например, проверить последнюю на данный момент)


Следуя соглашению о Семантическом Версионировании, любые версии ниже `1.0.0` потенциально могут добавить обратно несовместимые изменения.

FastAPI следует соглашению в том, что любые изменения "ПАТЧ"-версии предназначены для исправления багов и обратно совместимых изменений.
Copy link
Contributor

Choose a reason for hiding this comment

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

Может быть так лучше? "для исправления багов и внесения обратно совместимых изменений"


После создания тестов вы можете обновить свою версию **FastAPI** до более новой. После это следует убедиться, что ваш код работает корректно, запустив тесты.

Если все работает корректно или после внесения необходимых изменений все ваши тесты проходят, только тогда вы можете закрепить вашу новую версию `fastapi`.
Copy link
Contributor

Choose a reason for hiding this comment

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

(запятая) все работает корректно, или после внесения необходимых изменений все ваши тесты проходят


Разные версии **FastAPI** будут использовать более новые версии Starlette.

Так что решение о используемой версии Starlette, вы можете оставить **FastAPI**.
Copy link
Contributor

Choose a reason for hiding this comment

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

об используемой

@github-actions
Copy link
Contributor

📝 Docs preview for commit f184d09 at: https://6300753c1f6b4b0af3bc6edd--fastapi.netlify.app

Copy link
Contributor

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

В целом хорошо. Небольшие косметические правки и пара непереведённых слов.

fastapi==0.45.0
```

что значит, что вы будете использовать именно версию `0.45.0`.
Copy link
Contributor

Choose a reason for hiding this comment

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

что значит, что

не очень красиво с такими повторами...

  • Это означает
  • Сие гласит

fastapi>=0.45.0,<0.46.0
```

что значит, что вы используете версии `0.45.0` или выше, но меньше чем `0.46.0`. Например, версия `0.45.2` все еще будет подходить.
Copy link
Contributor

Choose a reason for hiding this comment

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

что значит, что

не очень красиво с такими повторами...

  • Это означает
  • Сие гласит


Первым делом вам следует "закрепить" конкретную последнюю используемую версию **FastAPI**, которая корректно работает с вашим приложением.

Например, вы используете версию `0.45.0` в приложении.
Copy link
Contributor

Choose a reason for hiding this comment

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

Например, вы используете версию 0.45.0 в приложении.

Например, в своём приложении.вы используете версию 0.45.0.


что значит, что вы используете версии `0.45.0` или выше, но меньше чем `0.46.0`. Например, версия `0.45.2` все еще будет подходить.

Если вы используете любой другой инструмент для организации установки, например Poetry, Pipenv или др., у них у всех имеется способ определения специфической версии для ваших пакетов.
Copy link
Contributor

Choose a reason for hiding this comment

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

для организации установки,

Мне чаще встречается - "управление зависимостями".


## Доступные версии

Вы можете увидеть доступные версии (например, проверить последнюю на данный момент) в [Release Notes](../release-notes.md){.internal-link target=_blank}.
Copy link
Contributor

Choose a reason for hiding this comment

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

увидеть

Посмотреть


FastAPI следует соглашению в том, что любые изменения "ПАТЧ"-версии предназначены для исправления багов и внесения обратно совместимых изменений.

!!! tip
Copy link
Contributor

Choose a reason for hiding this comment

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

Не переведено.

Подсказка/уточнение

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Я просто решил сохранить единообразие. В "Введение в аннотации типов" перевод подсказок и иных сносок отсутствует.
Снимок экрана от 2022-08-23 18-52-32

Copy link
Contributor

Choose a reason for hiding this comment

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

@Xewus если я правильно помню, !!! tip преобразуется в такой блок с подсказкой, если перевести, то сломается

Copy link
Contributor

@Xewus Xewus Aug 23, 2022

Choose a reason for hiding this comment

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

Возможно, Вы правы, завтра постараюсь пощупать этот случай.
Но вообще, чтобы проверить - сломается или нет, к фреймворку прилагаются инструменты для проверки локально - как будет реально выглядеть ваш перевод.
В целом, по данному переводу у меня особых претензий нет, я лишь предложил некоторые варианты, которые, по моему скромному мнению, звучат более "русскоязычно", потому и не нажимал "Требуются изменения".

Copy link
Contributor

Choose a reason for hiding this comment

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

@Xewus если я правильно помню, !!! tip преобразуется в такой блок с подсказкой, если перевести, то сломается

У корейцев работает.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Завтра проверю. Если локально проблем не будет, сделаю необходимые изменения.


Обратно несовместимые изменения и новые функции добавляются в "МИНОРНЫЕ" версии.

!!! tip
Copy link
Contributor

Choose a reason for hiding this comment

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

Не переведено.

Подсказка/уточнение

!!! tip
"МИНОРНАЯ" версия - это число в середине. Например, в `0.2.3` МИНОРНАЯ версия - это `2`.

## Обновление версии FastAPI
Copy link
Contributor

Choose a reason for hiding this comment

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

версии

Й


Вам следует добавить тесты для вашего приложения.

С помощью **FastAPI** это очень просто (благодаря Starlette), см. документацию: [Testing](../tutorial/testing.md){.internal-link target=_blank}
Copy link
Contributor

Choose a reason for hiding this comment

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

Testing

Не переведено.

@@ -57,6 +57,8 @@ nav:
- uk: /uk/
- zh: /zh/
- async.md
- Развёртывание:
- deployment/versions.md
Copy link
Contributor

Choose a reason for hiding this comment

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

Хорошо, что не забыли про этот файл.

@github-actions
Copy link
Contributor

📝 Docs preview for commit 9ff9f92 at: https://63063d19a9359c00680c8f92--fastapi.netlify.app

@emp7yhead
Copy link
Contributor Author

@Xewus @Winand, я исправил указанные замечания.
Блоки с подсказками и вправду без проблем переводятся, проверил локально. Может быть в рамках этого PR и на остальных страницах сделать перевод названий блоков?

@Xewus
Copy link
Contributor

Xewus commented Aug 24, 2022

@emp7yhead , когда пытался перевести что-то, Себастьян попросил делать отдельный ПР на каждую страницу, соответственно, на исправления тоже лучше отдельный ПР делать.


Часто добавляются новые функции, регулярно исправляются баги, код продолжает постоянно совершенствоваться.

По указанным причинам текущие версии до сих пор `0.x.x`. Это говорит о том, что каждая версия может содержать обратно несовместимые изменения, следуя <a href="https://semver.org/" class="external-link" target="_blank">соглашению о Семантическом Версионировании</a>.
Copy link
Contributor

@Winand Winand Aug 24, 2022

Choose a reason for hiding this comment

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

На русском, как будто бы, глагола не достаёт: "текущие версии до сих пор 0.x.x". Может быть "текущие версии до сих пор не достигли 1.x.x", раз речь о семвере? Впрочем, если над этим слишком долго думать, то выйдет уже 1.0 .))

@tiangolo
Copy link
Owner

Great, thanks @emp7yhead! 🚀 🍰

Thanks for the reviews @Winand and @Xewus! ☕

@tiangolo tiangolo merged commit fa92dbf into tiangolo:master Aug 26, 2022
@emp7yhead emp7yhead deleted the add-translation-ru-deployment-versions branch August 28, 2022 13:17
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

4 participants