[07:55:04]
<Yunohost Git/Infra notifications> [apps] @tituspijean pushed 1 commit to deprecate-rportd: Rportd has been closed-sourced and its repository deleted ([c65d421a](https://github.com/YunoHost/apps/commit/c65d421ae0726d7471229ff0d02a78edff751142))
[07:55:04]
<Yunohost Git/Infra notifications> [apps] @tituspijean created new branch deprecate-rportd
[07:56:27]
<Yunohost Git/Infra notifications> [apps] @tituspijean opened [pull request #1740](https://github.com/YunoHost/apps/pull/1740): Rportd has been closed-sourced and its repository deleted
[08:39:35]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1740](https://github.com/YunoHost/apps/pull/1740#pullrequestreview-1641022878) Rportd has been closed-sourced and its repository deleted
[08:39:40]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 2 commits to master ([102858c4da36...466f4750fc49](https://github.com/YunoHost/apps/compare/102858c4da36...466f4750fc49))
[08:39:45]
<Yunohost Git/Infra notifications> [apps/master] Rportd has been closed-sourced and its repository deleted - tituspijean
[08:39:45]
<Yunohost Git/Infra notifications> [apps] @ericgaspar deleted branch deprecate-rportd
[08:39:48]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1740 from YunoHost/deprecate-rportd Rportd has been closed-sourced and its repository deleted - eric_G
[08:40:28]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1738](https://github.com/YunoHost/apps/pull/1738): Update app levels according to CI results
[08:41:00]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([e421791c](https://github.com/YunoHost/apps/commit/e421791ca840dbc65f45a16f2ea86d32a0f1909e))
[08:41:52]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1738](https://github.com/YunoHost/apps/pull/1738): Update app levels according to CI results
[08:42:08]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([f10b93ab](https://github.com/YunoHost/apps/commit/f10b93ab8bcfd58ccbc0aa299f141ebacc078ae8))
[08:42:37]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1738](https://github.com/YunoHost/apps/pull/1738#pullrequestreview-1641023210) Update app levels according to CI results
[11:20:09]
<Yunohost Git/Infra notifications> App peertube failed all tests in job [#18803](https://ci-apps.yunohost.org/ci/job/18803) :(
[11:22:29]
<Salamandar> > Have postgres listed as apt dependency
Yeah that's part really logical, part a bit annoying
[11:23:11]
<Salamandar> Also I'm really wondering about depending on *implementations* of the psql/mysql protocol, instead of just asking yunohost "please install any valid implementation"
[11:23:33]
<Salamandar> but as today there's none other postgre implementation, I think it's okay 😄
[11:28:23]
<orhtej2> > <@Salamandar:matrix.org> Also I'm really wondering about depending on *implementations* of the psql/mysql protocol, instead of just asking yunohost "please install any valid implementation"
I believe recent `ghost` situation indicates that mysql proper is the only mysql-compatible implementation 😜
[11:28:43]
<Salamandar> yeah, you're right…
[12:43:16]
<Yunohost Git/Infra notifications> [apps] @Salamandar opened [pull request #1741](https://github.com/YunoHost/apps/pull/1741): Fix manifest.toml schema : add antifeatures section
[12:54:37]
<Yunohost Git/Infra notifications> [apps] @tituspijean approved [pull request #1741](https://github.com/YunoHost/apps/pull/1741#pullrequestreview-1641047595) Fix manifest.toml schema : add antifeatures section
[12:54:45]
<Yunohost Git/Infra notifications> [apps] @tituspijean pushed 2 commits to master ([466f4750fc49...940dce2b77e4](https://github.com/YunoHost/apps/compare/466f4750fc49...940dce2b77e4))
[12:54:45]
<Yunohost Git/Infra notifications> [apps] @tituspijean merged [pull request #1741](https://github.com/YunoHost/apps/pull/1741): Fix manifest.toml schema : add antifeatures section
[12:54:49]
<Yunohost Git/Infra notifications> [apps/master] Fix manifest.toml schema : add antifeatures section - Félix Piédallu
[12:54:54]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1741 from Salamandar/fix_manifest_schema_antifeatures Fix manifest.toml schema : add antifeatures s... - tituspijean
[12:57:42]
<Salamandar> Hmmm. Another issue in the schema is `pattern` for questions. And it looks like it doesn't work properly
[12:57:49]
<Salamandar> (or at all ?)
[12:58:07]
<Salamandar> for example : https://github.com/YunoHost-Apps/mautrix_whatsapp_ynh/blob/8f3f65f5cba8b5dc6ee53bfd62ffe149b3cfcf8a/manifest.toml#L31
[12:59:10]
<Salamandar> https://aria.im/_matrix/media/v1/download/matrix.org/dRgJRHKKkYkpUzoxCmOysJbT
[12:59:24]
<Salamandar> And here's the result in the install panel (cropped to hide my domain 😛 )
[13:00:05]
<tituspijean> It's now official: all apps are Synapse. *synapps*
[13:00:15]
<Salamandar> so it pre-selects `synapse` because the manifest provides a sane defalut value. But the pattern filter doesn't work ☹️
[13:01:43]
<Salamandar> Ah yeah ok I understand, it's a post-selection validation pattern. It says `Invalid app selected. Please select a Synapse instance.` when clicking on Install with an invalid app.
[13:01:49]
<Salamandar> Soooo it works, but not as I expected 😄
[13:04:15]
*Salamandar writes an issue about that
[14:11:05]
<Yunohost Git/Infra notifications> [apps] @Salamandar opened [pull request #1742](https://github.com/YunoHost/apps/pull/1742): Fix manifest schema
[14:18:57]
<Yunohost Git/Infra notifications> [apps] @alexAubin [commented](https://github.com/YunoHost/apps/pull/1742#issuecomment-1732583650) on [issue #1742](https://github.com/YunoHost/apps/pull/1742) Fix manifest schema: Cheers, didnt review all the stuff carefully but looks legit and blindly trusting :+1:
[14:19:01]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 5 commits to master ([940dce2b77e4...02d8b59f4354](https://github.com/YunoHost/apps/compare/940dce2b77e4...02d8b59f4354))
[14:19:01]
<Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #1742](https://github.com/YunoHost/apps/pull/1742): Fix manifest schema
[14:19:04]
<Yunohost Git/Infra notifications> [apps/master] schemas: Allow translated_string to be a simple string, the underlying code supports it. - Félix Piédallu
[14:19:21]
<Yunohost Git/Infra notifications> [apps/master] schemas: Add support install.*.pattern.{regex,error} - Félix Piédallu
[14:19:21]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1742 from Salamandar/fix_manifest_schema Fix manifest schema - Alexandre Aubin
[14:20:24]
<Yunohost Git/Infra notifications> [apps] @Salamandar opened [pull request #1743](https://github.com/YunoHost/apps/pull/1743): schemas: Add schemas for apps.toml and categories.toml
[14:23:21]
<Yunohost Git/Infra notifications> [apps] @alexAubin [commented](https://github.com/YunoHost/apps/pull/1743#issuecomment-1732584548) on [issue #1743](https://github.com/YunoHost/apps/pull/1743) schemas: Add schemas for apps.toml and categories.toml: Hmmmyes indeed it would be pretty cool to have a github action with this check (theres already one but only checks that...
[15:37:41]
<Yunohost Git/Infra notifications> [apps] @Salamandar [commented](https://github.com/YunoHost/apps/pull/1743#issuecomment-1732599188) on [issue #1743](https://github.com/YunoHost/apps/pull/1743) schemas: Add schemas for apps.toml and categories.toml: I think that json-schema can even incorporate some custom logic that can replace the catalog_linter.py (but is it what w...
[15:55:25]
<Yunohost Git/Infra notifications> [apps] @alexAubin [commented](https://github.com/YunoHost/apps/pull/1743#issuecomment-1732602946) on [issue #1743](https://github.com/YunoHost/apps/pull/1743) schemas: Add schemas for apps.toml and categories.toml: Im re-reading the catalog linter and its only for : - checking the url of the app ends with {app_id}_ynh for consis...
[16:18:25]
<Yunohost Git/Infra notifications> [apps] @Salamandar [commented](https://github.com/YunoHost/apps/pull/1743#issuecomment-1732599188) on [issue #1743](https://github.com/YunoHost/apps/pull/1743) schemas: Add schemas for apps.toml and categories.toml: I think that json-schema can even incorporate some custom logic that can replace the catalog_linter.py (but is it what w...
[16:24:34]
<Yunohost Git/Infra notifications> [apps] @Salamandar [commented](https://github.com/YunoHost/apps/pull/1743#issuecomment-1732613126) on [issue #1743](https://github.com/YunoHost/apps/pull/1743) schemas: Add schemas for apps.toml and categories.toml: Actually this was more advance in my memory than it is really, so, we can check its .*_ynh but not {app}_ynh, so th...
[17:46:20]
<Yunohost Git/Infra notifications> [apps] @Salamandar opened [pull request #1744](https://github.com/YunoHost/apps/pull/1744): Rework the catalog_linter.py
[17:46:44]
<Yunohost Git/Infra notifications> [apps] @Salamandar [commented](https://github.com/YunoHost/apps/pull/1744#issuecomment-1732629038) on [issue #1744](https://github.com/YunoHost/apps/pull/1744) Rework the catalog_linter.py: The jsonschema part depends on https://github.com/YunoHost/apps/pull/1743 obviously :D
[18:04:03]
<Aleks (he/him/il/lui)> > <@yunohostinfra:matrix.org> [apps] @Salamandar opened [pull request #1744](https://github.com/YunoHost/apps/pull/1744): Rework the catalog_linter.py
`APPS_ROOT = Path(__file__).parent.parent` my god i really need to start using `pathlib`
[18:05:00]
<Salamandar> yes agreed
[18:05:00]
<Aleks (he/him/il/lui)> i'm always like `os.path.dirname(__file__)` or i never remember wtf is the syntax
[18:05:07]
<Salamandar> also a = Path(...) ; a.open().read() ; a.touch() , a.exists() , …
[18:06:45]
<Aleks (he/him/il/lui)> and `path / "foo" / "bar.txt"`
[18:08:04]
<Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #1743](https://github.com/YunoHost/apps/pull/1743): schemas: Add schemas for apps.toml and categories.toml
[18:08:04]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 2 commits to master ([02d8b59f4354...8d4ecca7d3a5](https://github.com/YunoHost/apps/compare/02d8b59f4354...8d4ecca7d3a5))
[18:08:08]
<Yunohost Git/Infra notifications> [apps/master] schemas: Add schemas for apps.toml and categories.toml - Félix Piédallu
[18:08:13]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1743 from Salamandar/apps_schema schemas: Add schemas for apps.toml and categories.toml - Alexandre Aubin
[18:08:21]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 3 commits to master ([8d4ecca7d3a5...0b4dff90e699](https://github.com/YunoHost/apps/compare/8d4ecca7d3a5...0b4dff90e699))
[18:08:24]
<Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #1744](https://github.com/YunoHost/apps/pull/1744): Rework the catalog_linter.py
[18:08:26]
<Yunohost Git/Infra notifications> [apps/master] Rework the catalog_linter.py: * chmod +x * split into functions * add python typing * Now run jsonschema on apps.toml * ... - Félix Piédallu
[18:08:32]
<Yunohost Git/Infra notifications> [apps/master] Install jsonschema on github actions for catalog linter - Félix Piédallu
[18:08:35]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1744 from Salamandar/rework_catalog_linter Rework the catalog_linter.py - Alexandre Aubin
[18:11:54]
<Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #1738](https://github.com/YunoHost/apps/pull/1738): Update app levels according to CI results
[18:11:55]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 6 commits to master ([0b4dff90e699...a86cec66fbe1](https://github.com/YunoHost/apps/compare/0b4dff90e699...a86cec66fbe1))
[18:11:59]
<Yunohost Git/Infra notifications> [apps/master] Update apps.toml - Éric Gaspar
[18:12:05]
<Yunohost Git/Infra notifications> [apps] @alexAubin deleted branch update_app_levels
[18:12:05]
<Yunohost Git/Infra notifications> [apps/master] Update apps.toml - Éric Gaspar
[18:12:05]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1738 from YunoHost/update_app_levels Update app levels according to CI results - Alexandre Aubin
[18:14:21]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to master: ci: also trigger jobs on push ? ([d4d74e47](https://github.com/YunoHost/apps/commit/d4d74e47add8541a07aba2bf5c183ae7726cdcb2))
[18:24:20]
<Aleks (he/him/il/lui)> Salamandar: hmmm le job échoue parce que y'a des subtags qui existent pas ... je sais pas quoi en penser, pour moi ça peut etre okay-ish de rajouter des tags meme si ils existent pas stricto-sensu sur la catégorie et sont pas afficher dans la webadmin ... meme si ça peut etre aussi une typo / erreur ...
[18:24:21]
<Aleks (he/him/il/lui)> https://github.com/YunoHost/apps/actions/runs/6291650040/job/17080229744
[18:25:03]
<Salamandar> ah oui tu as déjà mergé ? ah 😲
[18:25:25]
<Salamandar> ouais du coup je sais pas trop
[18:25:35]
<Salamandar> on dirait qu'il y a des subtags qui ne sont pas dans la catégorie
[18:25:44]
<Aleks (he/him/il/lui)> 'fin j'ai un peu la flemme de réfléchir au sujet, je cacherais bien le probleme/question existentielle sous le tapis haha
[18:25:49]
<Aleks (he/him/il/lui)> yep
[18:26:25]
<Salamandar> genre social-media forum qui échoue, c'est parce que forum est dans communication
[18:26:40]
<Salamandar> peut-être qu'il faudrait accepter plusieurs catégories en fait…
[18:26:48]
<Aleks (he/him/il/lui)> hmpf
[18:27:25]
<Salamandar> ^^
[18:27:40]
<Salamandar> je peux virer ça en warning déjà
[18:28:21]
<Salamandar> > <@Alekswag:matrix.org> `APPS_ROOT = Path(__file__).parent.parent` my god i really need to start using `pathlib`
Also, typing 😄
[18:33:54]
<Yunohost Git/Infra notifications> [apps] @Salamandar opened [pull request #1745](https://github.com/YunoHost/apps/pull/1745): catalog linter: Pass subtags errors as warnings only
[18:37:41]
<Aleks (he/him/il/lui)> > <@Salamandar:matrix.org> Also, typing 😄
like do you have a tool somewhere that runs mypy for you ?
[18:39:14]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 2 commits to master ([d4d74e47add8...77c69998e96e](https://github.com/YunoHost/apps/compare/d4d74e47add8...77c69998e96e))
[18:39:14]
<Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #1745](https://github.com/YunoHost/apps/pull/1745): catalog linter: Pass subtags errors as warnings only
[18:39:18]
<Yunohost Git/Infra notifications> [apps/master] Pass subtags errors as warnings only - Félix Piédallu
[18:39:22]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1745 from Salamandar/rework_catalog_linter catalog linter: Pass subtags errors as warnings only - Alexandre Aubin
[18:47:23]
<Salamandar> > <@Alekswag:matrix.org> like do you have a tool somewhere that runs mypy for you ?
yes, my IDE
[20:11:14]
<Yunohost Git/Infra notifications> [apps] @Salamandar opened [pull request #1746](https://github.com/YunoHost/apps/pull/1746): schemas: fix tests.v1, args can be booleans too
[20:12:51]
<Yunohost Git/Infra notifications> [apps] @Salamandar [commented](https://github.com/YunoHost/apps/pull/1746#issuecomment-1732659251) on [issue #1746](https://github.com/YunoHost/apps/pull/1746) schemas: fix tests.v1, args can be booleans too: I also pushed a commit that fixes catalog_linter… sorry ><
[20:15:19]
<Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #1746](https://github.com/YunoHost/apps/pull/1746): schemas: fix tests.v1, args can be booleans too
[20:15:20]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 3 commits to master ([77c69998e96e...b5ff2da9c8af](https://github.com/YunoHost/apps/compare/77c69998e96e...b5ff2da9c8af))
[20:15:30]
<Yunohost Git/Infra notifications> [apps/master] schemas: fix tests.v1, args can be booleans too - Félix Piédallu
[20:15:31]
<Yunohost Git/Infra notifications> [apps/master] catalog_linter: Fix missing is_fatal on yield - Félix Piédallu
[20:15:39]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1746 from Salamandar/fix_tests_schema schemas: fix tests.v1, args can be booleans too - Alexandre Aubin
[20:36:10]
<orhtej2> assuming this is the autoupdater code: https://github.com/YunoHost/apps/blob/466f4750fc49bf106763e7b504ae477db5b83910/tools/autoupdate_app_sources/autoupdate_app_sources.py#L388C55-L388C55
Can we make it reset `ynh` part? So that `2.6.2~ynh4` becomes `2.7.0~ynh1` rather than `2.7.0~ynh4`?
[20:36:39]
<orhtej2> trivial as in replace `m.group(3)` with `'~ynh1'` I believe
[20:39:10]
<Salamandar> Oh yeah good catch
[20:41:08]
<orhtej2> should I yolocommit? Not sure how to test this tool TBH
[20:46:03]
<Aleks (he/him/il/lui)> yup sure go ahead if the tweak is obvious
[20:47:08]
<Aleks (he/him/il/lui)> i you really want to test you could clone a random app with autoupdater configured, manually change the version in manifest to a dummy version, then run the autoupdater script on that folder to see if it correctly updates the manifest
[20:53:19]
<Salamandar> Did you ever try to make chatgp write yunohost scripts ?
[20:53:25]
<Salamandar> chatgpt*
[20:54:13]
<Salamandar> (i'm thinking about it because my cousine is asking for help because her work blocked chatgpt access and she uses it to correct her english emails and write her company's newsletter…
[20:56:31]
<Salamandar> OK i just tried, it's a complete disaster, it claims it knows yunohost packaging but writes scripts that don't ressemble standard packages in any way…
[20:59:52]
<orhtej2> > <@Salamandar:matrix.org> Did you ever try to make chatgp write yunohost scripts ?
someone reported they tried and it created gibberish
[21:00:15]
<Salamandar> ah ah
[21:00:21]
<Salamandar> can confirm
[21:00:28]
<Salamandar> it did not get better
[21:00:45]
<orhtej2> must have been before Great Room Upgrade (TM) of 2023
[21:11:34]
<Yunohost Git/Infra notifications> [apps] @orhtej2 opened [pull request #1747](https://github.com/YunoHost/apps/pull/1747): Reset YNH integration version to 1 on upstream version bumps
[21:11:48]
<orhtej2> > <@yunohostinfra:matrix.org> [apps] @orhtej2 opened [pull request #1747](https://github.com/YunoHost/apps/pull/1747): Reset YNH integration version to 1 on upstream version bumps
UNTESTED
[21:24:50]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 2 commits to master ([b5ff2da9c8af...a8441a0dfaa0](https://github.com/YunoHost/apps/compare/b5ff2da9c8af...a8441a0dfaa0))
[21:24:50]
<Yunohost Git/Infra notifications> [apps/master] Reset YNH integration version to 1 on upstream version bumps - orhtej2
[21:24:50]
<Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #1747](https://github.com/YunoHost/apps/pull/1747): Reset YNH integration version to 1 on upstream version bumps
[21:24:51]
<Aleks (he/him/il/lui)> testing is doubting
[21:24:51]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1747 from orhtej2/patch-1 Reset YNH integration version to 1 on upstream version bumps - Alexandre Aubin
[21:25:08]
<Aleks (he/him/il/lui)> https://www.pinterest.fr/pin/360006563939441679/
[21:26:32]
<Salamandar> commitstrip ❤️ ❤️
[21:27:19]
<Salamandar> Ça, c'est un truc français qui a réussi à réussir à l'étranger 😄
[21:43:43]
<lapineige> > <@Salamandar:matrix.org> OK i just tried, it's a complete disaster, it claims it knows yunohost packaging but writes scripts that don't ressemble standard packages in any way…
I'm curious to see the result if you give it example_ynh in the prompt 🤔
[21:47:41]
<lapineige> (wait, I made a generator for that :D)
[21:52:06]
<orhtej2> People feeling lucky can give slim integration of `umami` a go by **making a proper backup**, then running
```sh
sudo yunohost app upgrade umami -u https://github.com/orhtej2/umami_ynh/tree/slim_prod
```
This slims down install dir:
```sh
# du -sh *
4.0G umami
1.2G umami__2
```