-
Notifications
You must be signed in to change notification settings - Fork 385
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
Set russian language #1381
Set russian language #1381
Conversation
Write translate file for Russian language
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.
The PR LGTM, thank you for your contribution @akprof2000
There is a merge conflict and the source repo has been deleted. I'll try to find a workaround to it to get it merged.
<note /> | ||
</trans-unit> | ||
<trans-unit id="CommandNoArgumentProvided"> | ||
<source>No argument was provided for Command '{0}'.</source> | ||
<target state="new">No argument was provided for Command '{0}'.</target> | ||
<target state="new">Нет аргументов для команды '{0}'.</target> |
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.
nit: according to google translate it means "No argument for command". We are missing the "was provided"
<note /> | ||
</trans-unit> | ||
<trans-unit id="ExceptionHandlerHeader"> | ||
<source>Unhandled exception: </source> | ||
<target state="new">Unhandled exception: </target> | ||
<target state="new">Необработанная ошибка: </target> |
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.
nit: According to google translate ошибка
means "error". Would "исключение" be a better name for exception?
<note /> | ||
</trans-unit> | ||
<trans-unit id="HelpUsageOptionsTitle"> | ||
<source>[options]</source> | ||
<target state="new">[options]</target> | ||
<target state="new">[опция]</target> |
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.
We need plural (options)
<target state="new">[опция]</target> | |
<target state="new">[oпции]</target> |
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.
Changing to capital О?
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.
@KalleOlaviNiemitalo great catch!
I am sorry, I've tried solving the merge conflicts but it seems impossible because of the fact the the repo was removed. |
Write translate file for Russian language