Skip to content

Translation codebase overview - #427

Merged
gcor merged 6 commits into
reactjs:masterfrom
BogdanDor:codebase-overview-translate
Feb 6, 2020
Merged

Translation codebase overview#427
gcor merged 6 commits into
reactjs:masterfrom
BogdanDor:codebase-overview-translate

Conversation

@BogdanDor

Copy link
Copy Markdown
Contributor

No description provided.

@netlify

netlifyBot commented Nov 4, 2019

Copy link
Copy Markdown

Deploy preview for ru-reactjs ready!

Built with commit d9358d9

https://deploy-preview-427--ru-reactjs.netlify.com

@BogdanDorBogdanDor mentioned this pull request Nov 4, 2019
88 tasks

@angryermineangryermine left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Привет, исправление опечаток

Comment threadcontent/docs/codebase-overview.md Outdated
Comment threadcontent/docs/codebase-overview.md Outdated
Comment threadcontent/docs/codebase-overview.md Outdated
Comment threadcontent/docs/codebase-overview.md Outdated
Comment threadcontent/docs/codebase-overview.md Outdated
Comment threadcontent/docs/codebase-overview.md Outdated
Comment threadcontent/docs/codebase-overview.md Outdated
Comment threadcontent/docs/codebase-overview.md Outdated
@ntishkevichntishkevich added the needs review A pull request ready to be reviewed label Nov 22, 2019
Co-Authored-By: Leonid Fedorov <lfedorov@live.com>

@gcorgcor left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Спасибо большое за перевод. 👍
Предлагаю такие улучшения:

Comment threadcontent/docs/codebase-overview.md Outdated
Comment threadcontent/docs/codebase-overview.md Outdated
Comment threadcontent/docs/codebase-overview.md Outdated
Если вы хотите [внести вклад в React](/docs/how-to-contribute.html), то мы надеемся, что эта глава вам в этом поможет.

We don't necessarily recommend any of these conventions in React apps. Many of them exist for historical reasons and might change with time.
Мы считаем, что вам не обязательно следовать этим соглашениям в ваших React приложениях. Многие из этих соглашений существуют по историческим причинам и могут быть изменены со временем.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
Мы считаем, что вам не обязательно следовать этим соглашениям в ваших React приложениях. Многие из этих соглашений существуют по историческим причинам и могут быть изменены со временем.
Мы не обязательно рекомендуем какие-либо из этих соглашений в приложениях React. Многие из этих соглашений существуют по историческим причинам и могут измениться со временем.

Comment threadcontent/docs/codebase-overview.md Outdated
Comment threadcontent/docs/codebase-overview.md Outdated
Comment threadcontent/docs/codebase-overview.md Outdated
Comment threadcontent/docs/codebase-overview.md Outdated
Comment threadcontent/docs/codebase-overview.md Outdated
Comment threadcontent/docs/codebase-overview.md Outdated
Comment threadcontent/docs/nav.yml Outdated
@gcor

gcor commented Jan 10, 2020

Copy link
Copy Markdown
Contributor

Возможно Reconsiler можно перевести как согласователь. раз reconciliation мы утвердили, как согласование

Co-Authored-By: Anton Ahatov <anton@gcor.ru>
@lex111

Copy link
Copy Markdown
Contributor

@BogdanDor давайте закончим перевод, осталось решить как перевести reconciler, @gcor верно указал на вариант "согласователь", как вам?

@BogdanDor

Copy link
Copy Markdown
ContributorAuthor

Термина reconciler нет в соглашении по переводу (файл TRANSLATION.md). Я предлагаю пока оставить reconciler, а в последующих коммитах уже добавить в глоссарий терминов перевод и сделать изменения в нужных местах.

@lex111lex111 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@BogdanDor давайте тогда переведем его как "согласователь", потому что "reconciliation" есть в соглашении, логично перевести это производное слово соответствующим образом.

Comment threadcontent/docs/codebase-overview.md Outdated
Если вы хотите [внести вклад в React](/docs/how-to-contribute.html), то мы надеемся, что эта глава вам в этом поможет.

We don't necessarily recommend any of these conventions in React apps. Many of them exist for historical reasons and might change with time.
Мы считаем, что вам не обязательно следовать этим соглашениям в ваших React приложениях. Многие из этих соглашений существуют по историческим причинам и могут быть изменены со временем.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
Мы считаем, что вам не обязательно следовать этим соглашениям в ваших Reactприложениях. Многие из этих соглашений существуют по историческим причинам и могут быть изменены со временем.
Мы считаем, что вам не обязательно следовать этим соглашениям в ваших React-приложениях. Многие из этих соглашений существуют по историческим причинам и могут быть изменены со временем.

Comment threadcontent/docs/codebase-overview.md Outdated
### Внешние зависимости {#external-dependencies}

React has almost no external dependencies. Usually, a `require()` points to a file in React's own codebase. However, there are a few relatively rare exceptions.
React, в основном, не имеет внеших зависимостей. Обычно `require()` указывает на файл, который находится в репозитории React. Однако существует несколько исключений.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
React, в основном, не имеет внеших зависимостей. Обычно `require()` указывает на файл, который находится в репозитории React. Однако существует несколько исключений.
React, в основном, не имеет внешних зависимостей. Обычно `require()` указывает на файл, который находится в репозитории React. Однако существует несколько исключений.

Comment threadcontent/docs/codebase-overview.md Outdated
React, в основном, не имеет внеших зависимостей. Обычно `require()` указывает на файл, который находится в репозитории React. Однако существует несколько исключений.

The [fbjs repository](https://github.com/facebook/fbjs) exists because React shares some small utilities with libraries like [Relay](https://github.com/facebook/relay), and we keep them in sync. We don't depend on equivalent small modules in the Node ecosystem because we want Facebook engineers to be able to make changes to them whenever necessary. None of the utilities inside fbjs are considered to be public API, and they are only intended for use by Facebook projects such as React.
Существует [репозиторий fbjs](https://github.com/facebook/fbjs), для того, чтобы React мог использовать общий код с библиотеками наподобии [Relay](https://github.com/facebook/relay). React не имеет зависимостей от модулей из экосистемы Node, для того, чтобы разработчики Facebook могли вносить изменения тогда, когда они им необходимы. fbjs не имеет внешнего API, а все его внутренние модули используются только проектами Facebook, например React.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Сменил фокус и уточнил.

Suggested change
Существует [репозиторий fbjs](https://github.com/facebook/fbjs), для того, чтобы React мог использовать общий код с библиотеками наподобии[Relay](https://github.com/facebook/relay). React не имеет зависимостей от модулей из экосистемы Node, для того, чтобы разработчики Facebook могли вносить изменения тогда, когда они им необходимы. fbjs не имеет внешнего API, а все его внутренние модули используются только проектами Facebook, например React.
Существует [репозиторий fbjs](https://github.com/facebook/fbjs) потому что React совместно использует небольшие вспомогательные функции и код с такими библиотеками как[Relay](https://github.com/facebook/relay). React не имеет зависимостей от модулей из экосистемы Node, чтобы разработчики Facebook могли вносить изменения тогда, когда они им необходимы. Никакая часть внутри fbjs не должна считаться за общедоступный API, поскольку он предназначен только для использования проектами Facebook, например React.

Comment threadcontent/docs/codebase-overview.md Outdated
### Каталоги верхнего уровня {#top-level-folders}

After cloning the [React repository](https://github.com/facebook/react), you will see a few top-level folders in it:
После копировния [репозитория React](https://github.com/facebook/react), вы увидите следующие верхнеуровневые каталоги:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
После копировния[репозитория React](https://github.com/facebook/react), вы увидите следующие верхнеуровневые каталоги:
После клонирования[репозитория React](https://github.com/facebook/react), на верхнем уровне вы увидите следующие каталоги:

Comment threadcontent/docs/codebase-overview.md Outdated
* [`fixtures`](https://github.com/facebook/react/tree/master/fixtures) contains a few small React test applications for contributors.
* `build` is the build output of React. It is not in the repository but it will appear in your React clone after you [build it](/docs/how-to-contribute.html#development-workflow) for the first time.
* [`packages`](https://github.com/facebook/react/tree/master/packages) содержит метаданные (такие как `package.json`) и исходный код (подкаталог `src`) для каждого пакета из репозитория React. **Если ваши изменения связаны с кодом, то подкаталог `src` внутри каждого пакета, это то место, где вы будете проводить большую часть времени.**
* [`fixtures`](https://github.com/facebook/react/tree/master/fixtures) содержит несколько небольших React приложений для контрибьютеров.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
*[`fixtures`](https://github.com/facebook/react/tree/master/fixtures) содержит несколько небольших Reactприложений для контрибьютеров.
*[`fixtures`](https://github.com/facebook/react/tree/master/fixtures) содержит несколько небольших React-приложений для контрибьюторов.

Comment threadcontent/docs/codebase-overview.md Outdated
### Тесты {#colocated-tests}

We don't have a top-level directory for unit tests. Instead, we put them into a directory called `__tests__` relative to the files that they test.
У нас нет отдельной верхнеуровневой директории для юнит тестов. Вместо этого, мы помещаем их в директорию `__tests__`, расположенную рядом с файлами, которые необходимо протестировать.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
У нас нет отдельной верхнеуровневой директории для юнит тестов. Вместо этого, мы помещаем их в директорию `__tests__`, расположенную рядом с файлами, которые необходимо протестировать.
У нас нет отдельной верхнеуровневой директории для модульных тестов. Вместо этого мы помещаем их в директорию `__tests__`, расположенную рядом с файлами, которые необходимо протестировать.

У нас нет отдельной верхнеуровневой директории для юнит тестов. Вместо этого, мы помещаем их в директорию `__tests__`, расположенную рядом с файлами, которые необходимо протестировать.

For example, a test for [`setInnerHTML.js`](https://github.com/facebook/react/blob/87724bd87506325fcaf2648c70fc1f43411a87be/src/renderers/dom/client/utils/setInnerHTML.js) is located in [`__tests__/setInnerHTML-test.js`](https://github.com/facebook/react/blob/87724bd87506325fcaf2648c70fc1f43411a87be/src/renderers/dom/client/utils/__tests__/setInnerHTML-test.js) right next to it.
Например, тесты для [`setInnerHTML.js`](https://github.com/facebook/react/blob/87724bd87506325fcaf2648c70fc1f43411a87be/src/renderers/dom/client/utils/setInnerHTML.js) расположены в [`__tests__/setInnerHTML-test.js`](https://github.com/facebook/react/blob/87724bd87506325fcaf2648c70fc1f43411a87be/src/renderers/dom/client/utils/__tests__/setInnerHTML-test.js).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
Например, тесты для [`setInnerHTML.js`](https://github.com/facebook/react/blob/87724bd87506325fcaf2648c70fc1f43411a87be/src/renderers/dom/client/utils/setInnerHTML.js)расположены в [`__tests__/setInnerHTML-test.js`](https://github.com/facebook/react/blob/87724bd87506325fcaf2648c70fc1f43411a87be/src/renderers/dom/client/utils/__tests__/setInnerHTML-test.js).
Например, тесты для [`setInnerHTML.js`](https://github.com/facebook/react/blob/87724bd87506325fcaf2648c70fc1f43411a87be/src/renderers/dom/client/utils/setInnerHTML.js)находятся в [`__tests__/setInnerHTML-test.js`](https://github.com/facebook/react/blob/87724bd87506325fcaf2648c70fc1f43411a87be/src/renderers/dom/client/utils/__tests__/setInnerHTML-test.js).

Comment threadcontent/docs/codebase-overview.md Outdated
```

Warnings are only enabled in development. In production, they are completely stripped out. If you need to forbid some code path from executing, use `invariant` module instead:
Предупреждения вкючены только в режиме разработки и полностью вырезаны из продакшена. Если вам необходимо запретить выполнение какого-либо кода, тогда используйте модуль `invariant`:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
Предупреждения вкючены только в режиме разработки и полностью вырезаны из продакшена. Если вам необходимо запретить выполнение какого-либо кода, тогда используйте модуль `invariant`:
Предупреждения отображаются только при разработке и полностью удалены из продакшен-сборки. Если вам необходимо запретить выполнение какого-либо кода, тогда используйте модуль `invariant`:

@BogdanDorBogdanDor left a comment

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Few changes

@BogdanDor
BogdanDor requested a review from lex111February 2, 2020 15:50

@lex111lex111 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@gcor посмотри ещё раз, пожалуйста, и если всё ок, то сливай этот пулреквест.

@gcor
gcor merged commit 8258309 into reactjs:masterFeb 6, 2020
@BogdanDor
BogdanDor deleted the codebase-overview-translate branch August 27, 2020 11:27
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs reviewA pull request ready to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@BogdanDor@gcor@lex111@angryermine@ntishkevich