-
-
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
A tool to help to backport locales, changes source strings to fix other broken translations #23633
A tool to help to backport locales, changes source strings to fix other broken translations #23633
Conversation
7436bb7
to
910036c
Compare
910036c
to
3641e10
Compare
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## main #23633 +/- ##
========================================
Coverage 47.14% 47.14%
========================================
Files 1149 1154 +5
Lines 151446 152353 +907
========================================
+ Hits 71397 71826 +429
- Misses 71611 72050 +439
- Partials 8438 8477 +39
... and 41 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
a3c67cd
to
2f05d3b
Compare
2f05d3b
to
65217ed
Compare
I think there's still a bug here. The Czech locale file in the referenced PR has ended up with an unclosed quote |
Two bugs:
They are all caused by the old update-locales.sh and the last restoring. But we can not import our ini to crowdin because they are still incompatible. I can do nothing at the moment because I do not have privileges. See the crowdin: |
# Conflicts: # options/locale/locale_el-GR.ini
Co-authored-by: delvh <dev.lh@web.de>
Co-authored-by: delvh <dev.lh@web.de>
Co-authored-by: delvh <dev.lh@web.de>
ping lg-tm |
* upstream/main: Fix incorrect `HookEventType` of pull request review comments (go-gitea#23650) [skip ci] Updated translations via Crowdin Fix codeblocks in the cheat sheet (go-gitea#23664) Drop migration for ForeignReference (go-gitea#23605) Fix new issue/pull request btn margin when it is next to sort (go-gitea#23647) A tool to help to backport locales, changes source strings to fix other broken translations (go-gitea#23633) Fix incorrect `show-modal` and `show-panel` class (go-gitea#23660) Restructure documentation. Now the documentation has installation, administration, usage, development, contributing the 5 main parts (go-gitea#23629) Check LFS/Packages settings in dump and doctor command (go-gitea#23631) Use a general approach to show tooltip, fix temporary tooltip bug (go-gitea#23574) Improve workflow event triggers (go-gitea#23613) Improve `<SvgIcon>` to make it output `svg` node and optimize performance (go-gitea#23570)
It use old en-US locales as reference, fill the old other locales with new locales.
More broken translations
Many translations are still broken. The reason is: at the last time restoring the ini to crowdin, many semicolon are treated as comments.
Two kinds of broken strings:
Some translations can be re-translated
So this PR also does some small changes on them, to trigger the re-translation.
The
locale_el-GR.ini
contains many broken tranlsationsI guess we should reset them from crowdin side, then translators can re-translate them.
Update: in latest main, the strings have been fixed.
TODO
Update: the el-GR translators have done great job and fixes these broken translations.
Merge this PR ASAP and upload
locale_el-GR.ini
to crowdin to remove broken strings.Out-dated, fixed in main.