Skip to content

Conversation

@ArduanovDanil
Copy link
Contributor

No description provided.

@ArduanovDanil ArduanovDanil marked this pull request as draft October 1, 2025 10:53
@samdark samdark requested a review from Copilot October 1, 2025 10:54
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR begins translating the concept/configuration guide from English to Russian. It creates a new Russian translation file and updates translation metadata files.

  • Adds complete Russian translation for the core concepts section about configuration
  • Updates PO files to mark the "Configuration" section as translated
  • Includes mixed content with both Russian and English sections in the configuration guide

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
guide/ru/views/view-injections.md New Russian view injections documentation with mixed language content
guide/ru/concept/configuration.md New Russian configuration guide with partial translation
_translations/guide/po/ru/views_view-injections.md.po Updated translation metadata for view injections
_translations/guide/po/ru/tutorial_using-yii-with-roadrunner.md.po Updated translation metadata for configuration section
_translations/guide/po/ru/tutorial_performance-tuning.md.po Updated translation metadata with timestamp changes
_translations/guide/po/ru/concept_configuration.md.po Updated translation metadata for configuration concept

Comment on lines +1 to +9
# View injections

The view injections are designed to provide a standardized way to pass
parameters to the common layer of views in an application. It allows
developers to manage the data that will be available across various views,
ensuring flexibility and reusability of code.

The view injections could be used if you require `yiisoft/yii-view-renderer`
package:
Copy link

Copilot AI Oct 1, 2025

Choose a reason for hiding this comment

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

The document title and introduction are in English while this is supposed to be a Russian translation file. All content should be consistently translated to Russian.

Copilot generated this review using guidance from repository custom instructions.
Comment on lines +69 to +94
There are many named configs defined. For each name, there is a
configuration.

A string means that the plugin takes config as is and merges it with
same-named configs from packages you require. That happens if these
packages have `config-plugin` in their `composer.json`.

The array means that the plugin will merge many files in the order they're
specified.

`?` at the beginning of the file path indicated that the file may be
absent. In this case, it's skipped.

`$` at the beginning of the name means a reference to another named config.

`params` is a bit special because it's reserved for application
parameters. These are automatically available as `$params` in all other
configuration files.

You can learn more about config plugin features [from its
documentation](https://github.com/yiisoft/config/blob/master/README.md).

## Config files

Now, as you know how the plugin assembles configs, look at `config`
directory:
Copy link

Copilot AI Oct 1, 2025

Choose a reason for hiding this comment

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

This large section is in English while the file is supposed to be a Russian translation. All content should be consistently translated to Russian according to the translation guidelines.

Copilot generated this review using guidance from repository custom instructions.
Comment on lines +121 to +126
The application consists of a set of services registered in a [dependency
container](di-container.md). The config files that responsible for direct
dependency container configuration are under `common/`, `console/` and
`web/` directories. We use `web/` for config specific to web application
and `console/` for config specific to console commands. Both web and console
are sharing configuration under `common/`.
Copy link

Copilot AI Oct 1, 2025

Choose a reason for hiding this comment

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

This section and the majority of content that follows is in English. The entire document should be consistently translated to Russian.

Copilot generated this review using guidance from repository custom instructions.
@ArduanovDanil ArduanovDanil marked this pull request as ready for review October 1, 2025 13:32
@ArduanovDanil ArduanovDanil marked this pull request as draft October 1, 2025 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant