Update spanish translations and fix syntax errors #4060

Merged
the-real-herowl merged 4 commits from :es-translations-syntax into master 2023-12-19 00:25:54 +01:00

Contribution to #3540

Currently running Wuzzy's Minetest_Translation_Tools in my branch gives this results:


TEST RESULT:

1037 file(s) checked. Test FAILED!
32 file(s) with error(s).
4 file(s) with warnings.
48 file(s) with empty translations.

I think that mods/ITEMS/mcl_honey/locale/mcl_honey.es.tr will be the last file that I'll fix the syntax errors for now.

Contribution to #3540 Currently running Wuzzy's Minetest_Translation_Tools in my branch gives this results: ------------ TEST RESULT: ------------ 1037 file(s) checked. Test FAILED! 32 file(s) with error(s). 4 file(s) with warnings. 48 file(s) with empty translations. I think that mods/ITEMS/mcl_honey/locale/mcl_honey.es.tr will be the last file that I'll fix the syntax errors for now.
megustanlosfrijoles added 1 commit 2023-12-10 06:54:33 +01:00
megustanlosfrijoles added 1 commit 2023-12-10 07:07:27 +01:00
Author
Member

One less warning with the last commit :)

the file was missing the #textdomain line

One less warning with the last commit :) the file was missing the #textdomain line
megustanlosfrijoles changed title from WIP: Update spanish translations and fix syntax errors to Update spanish translations and fix syntax errors 2023-12-10 07:08:30 +01:00
megustanlosfrijoles changed title from Update spanish translations and fix syntax errors to WIP: Update spanish translations and fix syntax errors 2023-12-10 07:09:06 +01:00
megustanlosfrijoles added 1 commit 2023-12-10 07:11:19 +01:00
megustanlosfrijoles added 1 commit 2023-12-10 07:13:52 +01:00
megustanlosfrijoles changed title from WIP: Update spanish translations and fix syntax errors to Update spanish translations and fix syntax errors 2023-12-10 07:17:55 +01:00
Author
Member

Ups, don't see those results that I posted earlier I think that I was running the script incorrectly, these are the results that I'm getting by running ../Minetest_Translation_Tools/mtt_check.py .


TEST RESULT:

1032 file(s) checked. Test FAILED!
27 file(s) with error(s).
1 file(s) with warnings.
47 file(s) with empty translations.

Ups, don't see those results that I posted earlier I think that I was running the script incorrectly, these are the results that I'm getting by running ../Minetest_Translation_Tools/mtt_check.py . ------------ TEST RESULT: ------------ 1032 file(s) checked. Test FAILED! 27 file(s) with error(s). 1 file(s) with warnings. 47 file(s) with empty translations.
the-real-herowl added the
needs discussion
translation
labels 2023-12-10 16:36:14 +01:00
Author
Member

Hi, what needs discussion?

Hi, what needs discussion?

Hi, what needs discussion?

The script used. We haven't checked how it works exactly yet, and I'd like to know that before we use it.

> Hi, what needs discussion? The script used. We haven't checked how it works exactly yet, and I'd like to know that before we use it.
Author
Member

Hi, what needs discussion?

The script used. We haven't checked how it works exactly yet, and I'd like to know that before we use it.

Sure. Well, if it's of any help these are the fixed errors, according to the script:

Error (./mods/ITEMS/mcl_fire/locale/mcl_fire.es.tr:15): Bad escape code in translation: '@ '
Error (./mods/ITEMS/mcl_fire/locale/mcl_fire.es.tr:15): Missing placeholder(s) in translation: '@1'

Warning (./mods/ITEMS/mcl_honey/locale/mcl_honey.es.tr): No textdomain in file

> > Hi, what needs discussion? > > The script used. We haven't checked how it works exactly yet, and I'd like to know that before we use it. Sure. Well, if it's of any help these are the fixed errors, according to the script: Error (./mods/ITEMS/mcl_fire/locale/mcl_fire.es.tr:15): Bad escape code in translation: '@ ' Error (./mods/ITEMS/mcl_fire/locale/mcl_fire.es.tr:15): Missing placeholder(s) in translation: '@1' Warning (./mods/ITEMS/mcl_honey/locale/mcl_honey.es.tr): No textdomain in file
Member

I think that the script is from a Minetest PR. (If that helps any)

I think that the script is from a Minetest PR. (If that helps any)
Author
Member

Maybe this is the information that you need?

MineClone2/MineClone2#3540 (comment)

Maybe this is the information that you need? https://git.minetest.land/MineClone2/MineClone2/issues/3540#issue-6272
the-real-herowl approved these changes 2023-12-19 00:25:02 +01:00
the-real-herowl removed the
needs discussion
label 2023-12-19 00:25:16 +01:00
the-real-herowl merged commit 99cc88ac42 into master 2023-12-19 00:25:54 +01:00
the-real-herowl added this to the 0.86.0 - Another Look milestone 2023-12-19 00:26:19 +01:00
megustanlosfrijoles deleted branch es-translations-syntax 2023-12-21 01:05:14 +01:00
Sign in to join this conversation.
No reviewers
No project
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: VoxeLibre/VoxeLibre#4060
No description provided.