Scripting assistance

[JaminShanti]

Hello everyone,
I recently submitted a pull request for the Windows deployment script. I would love to contribute further—are there any other scripts that need updating or refactoring? I am proficient in Bash, PowerShell, and Python, and I’m happy to help where needed.

[ann0see]

Thanks. Absolutely. For example #3597

And a lot of the stuff - especially for the website is in need for refactoring and updating.

[JaminShanti]

Ann, What did you need on the website?

[JaminShanti]

Thanks for pointing me to #3597, @ann0see!
​
Since the Python linter script is already fully drafted, am I correct that you primarily need a code review to get this over the finish line?

​If so, I can pull the branch locally to test the XML parsing and regex rules against src/translation to ensure there are no false positives. I'll also verify the exit code logic behaves exactly as expected for a CI pipeline quality gate.

​Let me know if I have that right, or if there is specific refactoring you need on this script. I’d be happy to look at the website repository next!

[ann0see]

I mean yes it's reviewing and potentially refactoring

[ann0see]

I think for the website it's mostly to get rid of too many external dependencies and making sure the build works on a recent Jekyll installation

[JaminShanti]

How is this output for #3597?

..\src\translation\translation_de_DE.ts line 174: de_DE: empty translation for '&OK...'
..\src\translation\translation_de_DE.ts line 205: de_DE: empty translation for 'Server...'
..\src\translation\translation_de_DE.ts line 358: de_DE: empty translation for 'PAN...'
..\src\translation\translation_de_DE.ts line 363: de_DE: empty translation for 'MUTE...'
..\src\translation\translation_de_DE.ts line 378: de_DE: empty translation for 'M...'
..\src\translation\translation_de_DE.ts line 383: de_DE: empty translation for 'S...'
..\src\translation\translation_de_DE.ts line 393: de_DE: empty translation for 'Alias/Name...'
..\src\translation\translation_de_DE.ts line 935: de_DE: empty translation for 'L...'
..\src\translation\translation_de_DE.ts line 940: de_DE: empty translation for 'R...'
..\src\translation\translation_de_DE.ts line 1428: de_DE: empty translation for 'Mono...'
..\src\translation\translation_de_DE.ts line 1459: de_DE: empty translation for 'Stereo...'
..\src\translation\translation_de_DE.ts line 1939: de_DE: empty translation for 'L...'
..\src\translation\translation_de_DE.ts line 1945: de_DE: empty translation for 'R...'
..\src\translation\translation_de_DE.ts line 2056: de_DE: empty translation for 'Auto...'
..\src\translation\translation_de_DE.ts line 2142: de_DE: empty translation for '%...'
..\src\translation\translation_de_DE.ts line 2962: de_DE: empty translation for 'STATUS...'
..\src\translation\translation_ja_JP.ts line 1175: ja_JP: leading/trailing whitespace mismatch for 'mode will increase your stream...'
..\src\translation\translation_ja_JP.ts line 1300: ja_JP: leading/trailing whitespace mismatch for 'Enabling...'
..\src\translation\translation_ja_JP.ts line 3116: ja_JP: leading/trailing whitespace mismatch for 'JACK was shut down. %1 require...'
..\src\translation\translation_pl_PL.ts line 144: pl_PL: empty translation for 'TextLabelVersion...'
..\src\translation\translation_th.ts line 845: th: leading/trailing whitespace mismatch for 'The delay is perfect for a jam...'
..\src\translation\translation_th.ts line 1015: th: leading/trailing whitespace mismatch for ', where...'
..\src\translation\translation_th.ts line 2060: th: leading/trailing whitespace mismatch for ', where...'
..\src\translation\translation_th.ts line 2166: th: leading/trailing whitespace mismatch for 'mode will increase your stream...'
..\src\translation\translation_zh_CN.ts line 1438: zh_CN: HTML missing for 'If you need to add additional ...'
Source: If you need to add additional directories to the Connect dialog Directory drop down, you can enter the addresses here.

Translation: 如果您需要在“连接”对话框的“目录”下拉列表中添加其他目录，您可以在这里输入地址

== Test Summary ==
[de_DE] Severe: 16, Warnings: 0
[es_ES] Severe: 0, Warnings: 0
[fr_FR] Severe: 0, Warnings: 0
[it_IT] Severe: 0, Warnings: 0
[ja_JP] Severe: 0, Warnings: 3
[ko_KR] Severe: 0, Warnings: 0
[nb_NO] Severe: 0, Warnings: 0
[nl_NL] Severe: 0, Warnings: 0
[pl_PL] Severe: 1, Warnings: 0
[pt_BR] Severe: 0, Warnings: 0
[pt_PT] Severe: 0, Warnings: 0
[ru_RU] Severe: 0, Warnings: 0
[sk_SK] Severe: 0, Warnings: 0
[sv_SE] Severe: 0, Warnings: 0
[th] Severe: 0, Warnings: 4
[zh_CN] Severe: 0, Warnings: 1

Total Severe: 17, Total Warnings: 8

Process finished with exit code 1

[ann0see]

Probably ok ish but looks messy still.

[JaminShanti]

How is this output? less findings after I rebased.

File: translation_ja_JP.ts
Line 1180 | WARNING | Leading/trailing whitespace mismatch for 'mode will increase your stream...'
Line 1305 | WARNING | Leading/trailing whitespace mismatch for 'Enabling'
Line 3470 | WARNING | Leading/trailing whitespace mismatch for 'JACK was shut down. %1 require...'

File: translation_th.ts
Line 845 | WARNING | Leading/trailing whitespace mismatch for 'The delay is perfect for a jam...'
Line 1015 | WARNING | Leading/trailing whitespace mismatch for ', where'
Line 2060 | WARNING | Leading/trailing whitespace mismatch for ', where'
Line 2166 | WARNING | Leading/trailing whitespace mismatch for 'mode will increase your stream...'

File: translation_zh_CN.ts
Line 1443 | WARNING | HTML missing for 'If you need to add additional ...'
| | Source: If you need to add additional directories to the Connect dialog Directory drop down, you can enter the addresses here.

| | Trans: 如果您需要在“连接”对话框的“目录”下拉列表中添加其他目录，您可以在这里输入地址

== Test Summary ==
[de_DE] Severe: 0, Warnings: 0
[es_ES] Severe: 0, Warnings: 0
[fr_FR] Severe: 0, Warnings: 0
[it_IT] Severe: 0, Warnings: 0
[ja_JP] Severe: 0, Warnings: 3
[ko_KR] Severe: 0, Warnings: 0
[nb_NO] Severe: 0, Warnings: 0
[nl_NL] Severe: 0, Warnings: 0
[pl_PL] Severe: 0, Warnings: 0
[pt_BR] Severe: 0, Warnings: 0
[pt_PT] Severe: 0, Warnings: 0
[ru_RU] Severe: 0, Warnings: 0
[sk_SK] Severe: 0, Warnings: 0
[sv_SE] Severe: 0, Warnings: 0
[th] Severe: 0, Warnings: 4
[zh_CN] Severe: 0, Warnings: 1

Total Severe: 0, Total Warnings: 8

Process finished with exit code 0

Also the top part is now color coated.