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

readme: minor changes + russian version #490

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pasabanov
Copy link

English

Changes in README.md

  1. Stylistic changes
  2. Made instructions for Chrome and Firefox clearer.

Issues:

  • By default, Chrome and Firefox do not provide NextDNS options. It might be worth clarifying the instructions or reconsidering this particular provider.

Russian version README.ru.md

Added a Russian version of the README.md file. The translation is approximately literal.

Issues:

  • The block with the utility instructions has not been translated.

Russian

Изменения в README.md

  1. Стилистические изменения
  2. Сделал инструкции для Chrome и Firefox более чёткими.

Проблемы:

  • По умолчанию Chrome и Firefox не предоставляют в вариантах выбора NextDNS. Возможно, стоит уточнить инструкции или пересмотреть выбор данного конкретного провайдера.

Русская версия README.ru.md

Добавлена русская версия файла README.md. Перевод приблизительно дословный.

Проблемы:

  • Не переведён блок с инструкцией по утилите.

@xu71ng
Copy link

xu71ng commented Aug 5, 2024

@CodiumAI-Agent /review

@CodiumAI-Agent
Copy link

PR Reviewer Guide 🔍

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Key issues to review

Clarity Improvement
The instructions for enabling 'Secure DNS' in Chrome and Firefox have been updated to provide more detailed paths. However, the description in the PR mentions that Chrome and Firefox do not provide NextDNS options by default. This might confuse users if NextDNS is not available as an option by default in these browsers. It would be beneficial to either provide additional guidance on how to add NextDNS or to clarify this point further to avoid user confusion.

Translation Inconsistency
The block with the utility instructions has not been translated into Russian. This inconsistency might confuse Russian-speaking users who expect the entire document to be in Russian. It is recommended to complete the translation for uniformity and to enhance the usability for Russian-speaking users.

@kitsuniru
Copy link

Качество перевода такое, что лучше оставить README на английском

@pasabanov
Copy link
Author

Качество перевода такое

Что не так с качеством?

@kitsuniru
Copy link

Качество перевода такое

Что не так с качеством?

Машинный перевод сомнительного качества

Оно работает с DPI, подключёнными с помощью оптического сплиттера или зеркалирования порта (Пассивное DPI), которые не блокируют данные, а просто отвечают быстрее, чем запрашиваемый пункт назначения, а также с Активным DPI, подключённым последовательно.

@pasabanov
Copy link
Author

Машинный перевод сомнительного качества

Ну можешь предложить формулировку получше.

@Jagerbar
Copy link

Jagerbar commented Aug 10, 2024

Машинный перевод сомнительного качества

Ну можешь предложить формулировку получше.

Просто закинь текст в Главред или другие для проверки пунктуации и синтаксиса. Тут не в формулировке дело, а в грамматических и синтаксических ошибках.

@pasabanov
Copy link
Author

Поправил формулировку. Теперь должно читаться лучше.

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.

5 participants