-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Backport locales to 1.19 #23634
Backport locales to 1.19 #23634
Conversation
Update: done |
4905b42
to
c865042
Compare
c865042
to
774b37b
Compare
1317c41
to
7c286d7
Compare
options/locale/locale_cs-CZ.ini
Outdated
@@ -221,7 +211,7 @@ invalid_db_setting=Nastavení databáze je neplatné: %v | |||
invalid_db_table=Databázová tabulka „%s“ je neplatná: %v | |||
invalid_repo_path=Kořenový adresář repozitářů není správný: %v | |||
invalid_app_data_path=Cesta k datům aplikace je neplatná: %v | |||
run_user_not_match="Run as" uživatelské jméno není aktuální uživatelské jméno: %s -> %s | |||
run_user_not_match="Run as\" uživatelské jméno není aktuální uživatelské jméno: %s -> %s |
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.
This appears to be missing an end quote?
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.
See the comment in #23633 (comment)
So I decided not to touch it.
And according to the behavior of current ini package usage, IIRC it doesn't need an end quote.
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.
If you prefer to manually fix these quotes, I can do it: in 72ec109 , there are a lot. I can't do more at the moment because I can not sync crowdin
Many of these strings are still broken in 1.20, due to the mess of last restoring.
Generated by #23633