[03:32:26]
<Yunohost Git/Infra notifications> [doc] dependabot[bot] pushed to dependabot/npm_and_yarn/mdast-util-to-hast-13.2.1: Bump mdast-util-to-hast from 13.2.0 to 13.2.1 Bumps [mdast-util-to-hast](https://github.com/syntax-tree/mdast-util-to-h... ([6438f396](https://github.com/YunoHost/doc/commit/6438f39657425e34cfbcba8757afc733825502fe))
[03:32:27]
<Yunohost Git/Infra notifications> [doc] dependabot[bot] opened [pull request #2711](https://github.com/YunoHost/doc/pull/2711): Bump mdast-util-to-hast from 13.2.0 to 13.2.1
[11:52:46]
<Yunohost Git/Infra notifications> [doc] yunohost-bot opened [pull request #2712](https://github.com/YunoHost/doc/pull/2712): Translations update from Weblate
[12:30:08]
<Yunohost Git/Infra notifications> [doc] alexAubin merged [pull request #2712](https://github.com/YunoHost/doc/pull/2712): Translations update from Weblate
[16:26:08]
<Yunohost Git/Infra notifications> [doc] Salamandar pushed to main: css: Fix fontawesome icons flashing when page loads ([dd0eac7a](https://github.com/YunoHost/doc/commit/dd0eac7a1f6aa4c80be78f8f993c3fc90aadaa65))
[16:29:33]
<Salamandar> aw crap doc is broken because of translations
[16:31:03]
<Yunohost Git/Infra notifications> [doc] Salamandar pushed to main: Upgrade Docusaurus and deps ([03343640](https://github.com/YunoHost/doc/commit/0334364085be344eaf44251e27b851839d3ad97f))
[16:31:09]
<Aleks (he/him/il/lui)> hmpf the ISP page is such a mess
[16:31:22]
<Salamandar> it builds locally so i don't get it
[16:31:53]
<Aleks (he/him/il/lui)> you probably need to regen the i18n file with po4a wiittth uuuh
[16:32:11]
<Aleks (he/him/il/lui)> https://github.com/YunoHost/doc/blob/main/.github/actions/docusaurus/action.yml#L39
[16:33:51]
<Salamandar> ho
[16:33:51]
<Salamandar> okay
[16:34:02]
<Salamandar> 'po4a_paths' is not defined in the configuration file. Where are the POT and PO files?
find: ‘i18n/*/docusaurus-plugin-content-docs/current/admin/’: Aucun fichier ou dossier de ce nom
[16:39:27]
<Aleks (he/him/il/lui)> uuuuh, gné ?
[16:40:34]
<Salamandar> well indeed those files don't exist
[16:41:07]
<Salamandar> not sure how this is supposed to work
[16:42:34]
<Aleks (he/him/il/lui)> the .po are in `i18n/docs/admin/`, the po4a script regenerate the .mdx using the .po
[16:42:48]
<Salamandar> but the issue seems to be that it doesn't read the config properly
[16:42:48]
<Aleks (he/him/il/lui)> what command are you running exactly and what's the full log
[16:42:53]
<Salamandar> python3 ./scripts/po4a.py build_translated_mdx
[16:43:01]
<Salamandar> ```
░▒▓█ python3 ./scripts/po4a.py build_translated_mdx
==========
[options] --master-language en --master-charset UTF-8 --localized-charset UTF-8 --package-name 'YunoHost documentation' --copyright-holder 'YunoHost Contributors'
[po4a_alias:markdown] text opt:"--option markdown --option breaks='import.*;|^\s*\<\S*\>\s*$|\s*<(div|img|YunoHostDocsCard|YunoHostDocsCardHeading|YunoHostImagesList|Tabs|InitialConfiguration|InstallScript|DocCardList).*>|\!\[\]\(/img/\S*\)' --option neverwrap --option nobullets --option yfm_keys=title,description,sidebar_label"
[po4a_langs] fr de it es
[po4a_paths] i18n/docs/admin/$master/en.pot $lang:i18n/docs/admin/$master/$lang.po
==========
'po4a_paths' is not defined in the configuration file. Where are the POT and PO files?
find: ‘i18n/*/docusaurus-plugin-content-docs/current/admin/’: Aucun fichier ou dossier de ce nom
```
[16:44:26]
<Aleks (he/him/il/lui)> hmmm, well it's supposed to iterate on every .mdx inside `docs/admin` ?
[16:44:45]
<Salamandar> yes but the issue is "'po4a_paths' is not defined in the configuration file."
[16:44:53]
<Salamandar> you're calling os.system………
[16:44:54]
<Aleks (he/him/il/lui)> ```
for page in sorted(Path("./docs/admin").rglob("*.mdx")):
page = str(page).split("/", 2)[-1] # Remove the starting 'docs/admin'
pot = page.replace("/", "__")[:-len(".mdx")]
conf += f'\n[type: markdown] ./docs/admin/{page} $lang:i18n/$lang/docusaurus-plugin-content-docs/current/admin/{page} pot:{pot} opt:"--keep 10"'
```
[16:45:02]
<Salamandar> so it doesn't catch the error codes
[16:45:45]
<Aleks (he/him/il/lui)> hmmmm i think that's a red herring, i've seen po4a being absolutely stupid like ... saying `po4a_paths` being not defined ... because there was less that 9 file entries (yeah ... )
[16:45:53]
<Salamandar> red herring ?
[16:45:58]
<Aleks (he/him/il/lui)> misdirection
[16:46:05]
<Salamandar> ah
[16:46:52]
<Salamandar> no
[16:46:52]
<Salamandar> it's not
[16:47:05]
<Salamandar> if i replace os.system with subprocess.check_call :
```
'po4a_paths' is not defined in the configuration file. Where are the POT and PO files?
Traceback (most recent call last):
File "/home/felix/tmp/yunohost/doc/./scripts/po4a.py", line 114, in <module>
main()
File "/home/felix/tmp/yunohost/doc/./scripts/po4a.py", line 91, in main
subprocess.check_call(f"{cmd} {po4a_conf.name} {flags}", shell=True)
File "/usr/lib/python3.12/subprocess.py", line 413, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command 'PERLLIB=./.po4a/lib/ ./.po4a/po4a /home/felix/tmp/yunohost/doc/po4a_a5k_tpto.cfg --no-update' returned non-zero exit status 255.
```
[16:48:32]
<Aleks (he/him/il/lui)> can you edit the script to dump the full configuration line 73ish because it purposefully doesnt dump all the `[type: markdown]` entries by default because there's a shitload of them
[16:49:44]
<Salamandar> hoooooooooooo
[16:49:49]
<Salamandar> adding `print(po4a_conf.read())` fixes the issue
[16:49:56]
<Salamandar> like if the file wasn't dumped
[16:50:00]
<Salamandar> ah yes
[16:50:03]
<Salamandar> you don't close the file…
[16:50:15]
<Salamandar> let me rewrite this script…
[16:54:30]
<Aleks (he/him/il/lui)> ah you mean flushing the write or something
[16:55:45]
<Salamandar> ye
[16:56:27]
<Salamandar> https://aria.im/_bifrost/v1/media/download/ARxQX_xnrdUzobydYrJop7dnVfEWGEp0AmjVaZEmUlbvMaL1ir68Eo-hylgcAlLudcyyRUrB7pu3bmmm3zcl7lxCea5STTkwAG1hdHJpeC5vcmcvV2VBTkVpY2VNRE1DcXJEbWtZZW5Qc096
[16:56:28]
<Salamandar> o noes
[16:58:07]
<Aleks (he/him/il/lui)> rofl
[17:29:48]
<Salamandar> Also uhm ?
[17:29:50]
<Salamandar> https://aria.im/_bifrost/v1/media/download/AYt-JFFZp_8ND2ML1jjIJJcy1Avz3P_0yWvz8PtW3-fl7YQPfi6SQ_X4HpFTyCjrh-nS5E-9akU-w1eAIYZgf5dCea5UNkQwAG1hdHJpeC5vcmcvSHFnWFdabklUSFhISlBSbHJzSmxOQWVo
[17:33:37]
<Aleks (he/him/il/lui)> yeah maybe should be named `regen_pot` whatever, that's because initially when writing the script, i didnt configure yet weblate which now takes care of updating the .po according to the .pot
[17:34:08]
<Aleks (he/him/il/lui)> though i don't even know if that's better to leave this to weblate in fact, because that creates huge boring PRs when it could be part of the po4a autocommit
[17:40:57]
<Salamandar> Okay i rename it
[17:50:20]
<Salamandar> should be good
[17:50:21]
<Yunohost Git/Infra notifications> [doc] Salamandar pushed to main: Refactor scritps/po4a.py ([51df6875](https://github.com/YunoHost/doc/commit/51df6875efa76c147bce46e6fd6cefb11c0344a8))
[17:50:25]
<Salamandar> crossing fingers that it doesn't break more
[17:51:13]
<Salamandar> that's the issue.
[17:51:14]
<Salamandar> https://aria.im/_bifrost/v1/media/download/AVaeMyqW96fN6bjPzqPzpXQaxs-bb7U1SvYupfSYD7Dy4m4dguvroDhowuJ71aaeEb_03DSwxPvmijI4EVJtplZCea5Vb6xAAG1hdHJpeC5vcmcvRFVWSHlFREpKbVlRUXdPRndzbVdOb2ly
[17:51:18]
<Salamandar> "french" quotes…
[17:51:34]
<Aleks (he/him/il/lui)> ah yes i actually fixed it on weblate earlier ...
[17:51:41]
<Salamandar> ha :)
[17:51:49]
<Salamandar> so do we just need to merge the pr ?
[17:51:55]
<Aleks (he/him/il/lui)> i can make it push or something
[17:52:14]
<Salamandar> if you can, that'd be great <3
[17:53:21]
<Aleks (he/him/il/lui)> `504 Gateway Time-out` 🫠
[17:53:42]
<Salamandar> yeye
[17:53:58]
<Salamandar> hmmm works for me ?
[17:54:21]
<Aleks (he/him/il/lui)> that's when i click the "push" button for it to push 1-2 commits and create a new PR
[17:54:29]
<Aleks (he/him/il/lui)> dunno what the heck could take so long
[17:54:36]
<Yunohost Git/Infra notifications> [doc] yunohost-bot opened [pull request #2713](https://github.com/YunoHost/doc/pull/2713): Translations update from Weblate
[17:55:14]
<Aleks (he/him/il/lui)> ah yeah idk it regenerated the po/pot again whatever
[17:55:33]
<Aleks (he/him/il/lui)> timestamp in .po file, sooooo important zzzz
[17:55:57]
<Aleks (he/him/il/lui)> an no software agrees on how stuff should be wrapped or not
[17:56:19]
<Aleks (he/him/il/lui)> anyway...
[17:56:24]
<Yunohost Git/Infra notifications> [doc] alexAubin merged [pull request #2713](https://github.com/YunoHost/doc/pull/2713): Translations update from Weblate
[17:56:43]
<Aleks (he/him/il/lui)> btw the "deploy preview" thingy keeps crashing when running on a PR from forks
[17:56:45]
<Yunohost Git/Infra notifications> [doc] Salamandar pushed to main: admin: Replace typographic quotes with basic quotes ([59291407](https://github.com/YunoHost/doc/commit/5929140788dd43ce60cdce728fb9f1c3470a71fc))
[17:56:51]
<Salamandar> hmmm
[17:56:52]
<Salamandar> ye that makes sense tbh
[17:57:08]
<Aleks (he/him/il/lui)> yeah...
[17:57:34]
<Salamandar> hmm
[17:57:38]
<Salamandar> do you have a link ?
[17:57:38]
<Aleks (he/him/il/lui)> but idk maybe there's a way to not run it on "foreign" PR or manually granting the rights for the PR to deploy a preview or idk
[17:57:47]
<Yunohost Git/Infra notifications> [doc] yunohost-bot opened [pull request #2714](https://github.com/YunoHost/doc/pull/2714): Translations update from Weblate
[17:58:28]
<Salamandar> yes i removed typographic quotes on another file
[17:58:31]
<Salamandar> (in the english version)
[17:58:35]
<Yunohost Git/Infra notifications> [doc] alexAubin merged [pull request #2714](https://github.com/YunoHost/doc/pull/2714): Translations update from Weblate
[18:00:08]
<Salamandar> welp
[18:00:23]
<Salamandar> the preview should live on the PR origin repository, not target, i guess
[18:00:44]
<Salamandar> cuz we can't have a foreign pr push on our repository
[18:00:52]
<Salamandar> (i mean maybe we can, but i refuse it)
[18:01:51]
<Salamandar> btw that's it, my company team now uses docusaurus
[18:02:00]
<Salamandar> courtesy of me
[18:02:47]
<Yunohost Git/Infra notifications> [doc] yunohost-bot opened [pull request #2715](https://github.com/YunoHost/doc/pull/2715): Translations update from Weblate
[18:02:57]
<Aleks (he/him/il/lui)> bruh
[18:04:53]
<Salamandar> wut
[18:04:54]
<Yunohost Git/Infra notifications> [doc] alexAubin merged [pull request #2715](https://github.com/YunoHost/doc/pull/2715): Translations update from Weblate
[18:04:56]
<Aleks (he/him/il/lui)> https://aria.im/_bifrost/v1/media/download/AUeFzqq_HGWGXvsDBirsUHwSscExmlks7kBPWCY8_UhI8u2A7NbdkNHQug69sYc8S18ogJV7d6pWhDlh8MqGKb1Cea5WOHFwAG1hdHJpeC5vcmcvTXFtVk1iVGxDSnZpSlpMa3hJSkZnR3dy
[18:04:57]
<Salamandar> huuuuum
[18:05:01]
<Aleks (he/him/il/lui)> stopitgetsomehelp.gif
[18:05:06]
<Salamandar> just date in po
[18:05:07]
<Salamandar> is it my fault
[18:05:08]
<Salamandar> does weblate run the script i just edited?
[18:05:32]
<Aleks (he/him/il/lui)> hmpf
[18:05:46]
<Aleks (he/him/il/lui)> no but github does and update the .pot : https://github.com/YunoHost/doc/commit/a5e01e86d246bc2e982af11624e517b13101b1ef
[18:05:58]
<Aleks (he/him/il/lui)> don't remember how but there's supposedly a mechanism to not commit stuff with only a single line diff
[18:07:19]
<Aleks (he/him/il/lui)> but also it feels like weblate shouldn't be the one taking care of the .pot -> .po update because that just creates so much noise in the PR diff ~\_~
[18:08:20]
<Yunohost Git/Infra notifications> [doc] yunohost-bot opened [pull request #2716](https://github.com/YunoHost/doc/pull/2716): Translations update from Weblate
[18:09:17]
<Aleks (he/him/il/lui)> https://www.youtube.com/watch?v=CAyWN9ba9J8&t=15s