Skip to content

Conversation

@seoeaa
Copy link

@seoeaa seoeaa commented Nov 18, 2023

added translation into Russian

added translation into Russian
checking yaml file wof validity
@shuashuai shuashuai self-assigned this Nov 20, 2023
Copy link
Member

@shuashuai shuashuai left a comment

Choose a reason for hiding this comment

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

Thank you very much for your contribution.

For yaml files, there are strict requirements for indentation. If you change the indentation here, it will cause language display errors.In addition, there are modifications to the indentation errors below. I just pointed out two errors.

Our translation work is connected to crowdin. If you are not a developer, we recommend you to go to crowdin-answer to contribute translation. Here you no longer need to consider the file format and only need to focus on the translation itself. If you have any questions about translation, feel free to contact me.

We look forward to your contribution.

i18n/ru_RU.yaml Outdated
desc:
other: This question currently includes multiple questions in one. It should focus on one problem only.
looks_ok:
reason:
Copy link
Member

Choose a reason for hiding this comment

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

Indentation error

i18n/ru_RU.yaml Outdated
other: "[{{.SiteName}}] {{.DisplayName}} answered your question"
body:
other: "<a href='{{.AnswerUrl}}'>{{.QuestionTitle}}</a><br><br>\n\n{{.DisplayName}}:<br>\n<blockquote>{{.AnswerSummary}}</blockquote><br>\n<a href='{{.AnswerUrl}}'>View it on {{.SiteName}}</a><br><br>\n\n--<br>\n<small><a href='{{.UnsubscribeUrl}}'>Unsubscribe</a></small>"
other: Этот пост требует другой причины, не указанной выше.
Copy link
Member

Choose a reason for hiding this comment

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

Indentation error

Copy link
Author

Choose a reason for hiding this comment

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

I fixed everything

@Justman100
Copy link

@shuashuai

Can simply closed

@seoeaa
Copy link
Author

seoeaa commented Dec 24, 2023

I'll fix it

@seoeaa
Copy link
Author

seoeaa commented Dec 24, 2023

I fixed everything

@seoeaa seoeaa requested a review from shuashuai December 24, 2023 14:40
@seoeaa
Copy link
Author

seoeaa commented Dec 24, 2023

checked the file

`import yamllint
import yamllint.config
import yamllint.linter

def lint_yaml(file_path):
with open(file_path, 'r', encoding='utf-8') as file:
content = file.read()
conf = yamllint.config.YamlLintConfig('extends: relaxed')
issues = yamllint.linter.run(input=content, conf=conf, filepath=file_path)
for issue in issues:
print(issue)
lint_yaml('ru_RU.yaml')`

@Justman100
Copy link

checked the file

`import yamllint
import yamllint.config
import yamllint.linter

def lint_yaml(file_path):
with open(file_path, 'r', encoding='utf-8') as file:
content = file.read()
conf = yamllint.config.YamlLintConfig('extends: relaxed')
issues = yamllint.linter.run(input=content, conf=conf, filepath=file_path)
for issue in issues:
print(issue)
lint_yaml('ru_RU.yaml')`

Isn't that managed by Crowdin?

@seoeaa
Copy link
Author

seoeaa commented Dec 24, 2023

Crowdin

no I used Python

@Justman100
Copy link

Crowdin

no I used Python

You change translations are managed by the Crowdin Project

@seoeaa
Copy link
Author

seoeaa commented Dec 24, 2023

Краудин

нет, я использовал Python

Вы меняете переводы, которые управляются проектом Crowdin.

then add this file to Crowdin

@Justman100
Copy link

Краудин

нет, я использовал Python

Вы меняете переводы, которые управляются проектом Crowdin.

then add this file to Crowdin

What for file??? You only changed existing translations! Which one only does in the editor of the Crowdin project!

@seoeaa
Copy link
Author

seoeaa commented Dec 24, 2023

Краудин

нет, я использовал Python

Вы меняете переводы, которые управляются проектом Crowdin.

затем добавьте этот файл в Crowdin

Что за файл??? Вы только изменили существующие переводы! Что только и делается в редакторе проекта Crowdin!

хорошо

@shuashuai
Copy link
Member

@seoeaa
I saw that someone has updated the Russian translation content on crowdin. Is this your contribution? If so, we will close this PR and give priority to the translation on crowdin!

@seoeaa
Copy link
Author

seoeaa commented Jan 1, 2024

@seoeaa I saw that someone has updated the Russian translation content on crowdin. Is this your contribution? If so, we will close this PR and give priority to the translation on crowdin!

yes

@Justman100
Copy link

Than this PR can be closed

@shuashuai shuashuai closed this Jan 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants