-
Notifications
You must be signed in to change notification settings - Fork 3
Conversation
based on f07fd3d commit by @samoylenko
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.
перевод супер! спасибо! мои коментарии - просто для информации, и если сразу после прочтения моих поментариев нет реакции "о, надо поменять", то и не надо ничего менять. уже отлично сделана работа, а я могу любые изменения предложить в отдельном PR.
Так же, этот PR замечателен тем, что он базирован не на ветке main
, а на моей, которая была еще не замерджена. Это отличное интро в тему 'rebase&squash'
docs/ru/getting-started.md
Outdated
| Microsoft Teams | ([ссылка](https://www.microsoft.com/en-us/microsoft-365/microsoft-teams/download-app)) | Клиент для проведения конференций. | | ||
| Git | ([ссылка](https://git-scm.com/)) | Система контроля версий<br/>**для пользователей Windows** - мы рекомендуем использовать Cygwin вместо отдельного установщика. | | ||
| NodeJS (текущий) | ([ссылка](https://nodejs.org/)) | Выбранная нами платформа. | | ||
| Docker | ([ссылка](https://www.docker.com/get-started)) | Runtime-контейнер<br/>**для пользователей Windows** рекомендуется Docker Desktop<br/>**для других ОС** - следуйте инструкции установки Docker в вашей системе. Кроме того, [`podman`](https://podman.io/) не будет лишним<br/><br/>Имей в виду: это требование может измениться, когда мы будем использовать [Kubernetes](https://kubernetes.io/) в наших проектах | |
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.
"Имей ввиду" -> "Имейте ввиду"? В остальных местах обращение везде на "вы"
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.
Ну вот, я еще про podman пропустил - он просто альтернатива docker на линуксе. Предлагаю принять изменения как есть, а я потом еще один PR предложу с моими добавлениями вместо исправлений этого. Задача перевода выполнена на отлично
docs/ru/getting-started.md
Outdated
| NodeJS (текущий) | ([ссылка](https://nodejs.org/)) | Выбранная нами платформа. | | ||
| Docker | ([ссылка](https://www.docker.com/get-started)) | Runtime-контейнер<br/>**для пользователей Windows** рекомендуется Docker Desktop<br/>**для других ОС** - следуйте инструкции установки Docker в вашей системе. Кроме того, [`podman`](https://podman.io/) не будет лишним<br/><br/>Имей в виду: это требование может измениться, когда мы будем использовать [Kubernetes](https://kubernetes.io/) в наших проектах | | ||
| Редактор кода ([IDE](https://en.wikipedia.org/wiki/Integrated_development_environment)) на ваш выбор | например [Intellij IDEA](https://www.jetbrains.com/idea/) или [Visual Studio Code](https://code.visualstudio.com/) | Дело вкусов, но вам нужен инструмент для редактирования кода. К слову, VS Code наиболее популярен среди NodeJS разработчиков. Но большая часть нашей команды предпочитает Intellij IDEA (или WebStorm, который де-факто тот же IDE) | | ||
| Cygwin (**Пользователи Windows**) | ([ссылка](https://cygwin.com/install.html))<br/>([package list](https://github.com/devsecops-learning/cygwin-scripts)) | POSIX оболочка и GNU инструментарий для Windows.<br/>Это рекомендуемый вариант установки `git` на Windows. | |
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.
Environment это скорее среда, а не оболочка. Но "оболочка" тоже отлично
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.
И мне кажется "метод установки" звучит более понятно чем "вариант установки", но тоже в твоем переводе супер. Этот и подобные коментарии - больше для информации. Если прям сильно надо будет что-то поменять, я в будущем просто другой PR сделаю
Взял на себя наглость добавить остальных членов команды в ревьюеры: сорян если в моем интерфейсе отобразилось неправильно. Что я увидел - нужны два аппрувала, а PR показывал только меня с уведомлением. Надо будет где-то в найстройках поправить |
Correct my mistakes noticed by @samoylenko
Исправил ошибки в |
Approved. Thank you. Если @sashablacky и @Fastex26rus сегодня не смогут посмотреть, можно попробовать договориться в телеге чтобы был либо 1 апрувер, либо узнать почему нотификации не проходят |
@radaken please feel free to merge now |
I'm translated to Russian language a
getting-started.md
and placed it to/ru/
folder when all Russian files are shall be. Please review that page and correct/comment misstakes if there are. I allowed for myself a free-translate (because I think the meaning persists, less-machine translate, and more human-friendly) and replace some links to ru-versions.