Sunday, May 19, 2024
dev@conference.yunohost.org
May
Mon Tue Wed Thu Fri Sat Sun
    1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
   
             

[11:50:22] <Yunohost Git/Infra notifications> [issues] chri2 opened [issue #2396](https://github.com/YunoHost/issues/issues/2396): improve ynh_local_curl
[11:50:22] <Yunohost Git/Infra notifications> [issues] chri2 labeled :birthday: feature on [issue #2396](https://github.com/YunoHost/issues/issues/2396): improve ynh_local_curl
[11:52:00] <Yunohost Git/Infra notifications> [issues] alexAubin labeled :package: App management on [issue #2393](https://github.com/YunoHost/issues/issues/2393): if an account app exists app will not install
[11:52:00] <Yunohost Git/Infra notifications> [issues] alexAubin labeled :cake: enhancement on [issue #2393](https://github.com/YunoHost/issues/issues/2393): if an account app exists app will not install
[11:52:04] <Yunohost Git/Infra notifications> [issues] alexAubin unlabeled :space_invader: bug on [issue #2393](https://github.com/YunoHost/issues/issues/2393): if an account app exists app will not install
[11:52:17] <Yunohost Git/Infra notifications> [issues] alexAubin edited [issue #2393](https://github.com/YunoHost/issues/issues/2393): if an account app exists app, use app__2 instead
[11:52:24] <Yunohost Git/Infra notifications> [issues] chri2 edited [issue #2396](https://github.com/YunoHost/issues/issues/2396): improve ynh_local_curl
[11:52:26] <Yunohost Git/Infra notifications> [issues] alexAubin edited [issue #2393](https://github.com/YunoHost/issues/issues/2393): if an account app exists when trying to install app, use app__2 instead
[11:53:30] <Yunohost Git/Infra notifications> [issues] alexAubin [commented](https://github.com/YunoHost/issues/issues/2386#issuecomment-2119208612) on [issue #2386](https://github.com/YunoHost/issues/issues/2386) Run tests without an actual Yunohost install: Closing because ynh-dev addresses the issue ...
[11:53:30] <Yunohost Git/Infra notifications> [issues] alexAubin closed [issue #2386](https://github.com/YunoHost/issues/issues/2386): Run tests without an actual Yunohost install
[11:54:39] <Yunohost Git/Infra notifications> [issues] alexAubin labeled :cake: enhancement on [issue #2383](https://github.com/YunoHost/issues/issues/2383): ynh_use_logrotate documentation misleading/wrong
[11:54:40] <Yunohost Git/Infra notifications> [issues] alexAubin labeled 🤓 helpers on [issue #2383](https://github.com/YunoHost/issues/issues/2383): ynh_use_logrotate documentation misleading/wrong
[11:54:40] <Yunohost Git/Infra notifications> [issues] alexAubin unlabeled :space_invader: bug on [issue #2383](https://github.com/YunoHost/issues/issues/2383): ynh_use_logrotate documentation misleading/wrong
[11:56:29] <Yunohost Git/Infra notifications> [issues] alexAubin [commented](https://github.com/YunoHost/issues/issues/2384#issuecomment-2119209346) on [issue #2384](https://github.com/YunoHost/issues/issues/2384) Bookworm: User cannot login to the portal before having access to an app: Fixed in https://github.com/YunoHost/yunohost/commit/5e406a55fa9a45069cb45093a915b0fe5fe18e10
[11:56:38] <Yunohost Git/Infra notifications> [issues] alexAubin closed [issue #2384](https://github.com/YunoHost/issues/issues/2384): Bookworm: User cannot login to the portal before having access to an app
[12:07:35] <Yunohost Git/Infra notifications> [issues] alexAubin [commented](https://github.com/YunoHost/issues/issues/2381#issuecomment-2119212638) on [issue #2381](https://github.com/YunoHost/issues/issues/2381) CI: comments in the manifest.toml are generating errors: Fixed via https://github.com/YunoHost/package_check/commit/aaca7137908169f04b6357bd18a89e2f6add6c41
[12:07:42] <Yunohost Git/Infra notifications> [issues] alexAubin closed [issue #2381](https://github.com/YunoHost/issues/issues/2381): CI: comments in the manifest.toml are generating errors
[12:12:18] <Yunohost Git/Infra notifications> [issues] chri2 [commented](https://github.com/YunoHost/issues/issues/2396#issuecomment-2119214199) on [issue #2396](https://github.com/YunoHost/issues/issues/2396) improve ynh_local_curl: about https://github.com/YunoHost/yunohost/pull/547 - 2 seconds sleep - _just in case_ Ill take this out of the sub fo...
[12:13:23] <Yunohost Git/Infra notifications> [yunohost] chri2 [commented](https://github.com/YunoHost/yunohost/pull/547#issuecomment-2119214570) on [issue #547](https://github.com/YunoHost/yunohost/pull/547) Fix http2 with curl: https://github.com/YunoHost/issues/issues/2396#issuecomment-2119214199
[12:17:08] <Yunohost Git/Infra notifications> [issues] alexAubin edited [issue #2388](https://github.com/YunoHost/issues/issues/2388): "domain dns push" doesnt work for googleclouddns
[12:17:28] <Yunohost Git/Infra notifications> [issues] alexAubin labeled :globe_with_meridians: DNS on [issue #2388](https://github.com/YunoHost/issues/issues/2388): "domain dns push" doesnt work for googleclouddns
[12:19:11] <Yunohost Git/Infra notifications> [issues] alexAubin [commented](https://github.com/YunoHost/issues/issues/2385#issuecomment-2119216322) on [issue #2385](https://github.com/YunoHost/issues/issues/2385) Use Custom Certificate Authority : Closing as wontfix
[12:19:11] <Yunohost Git/Infra notifications> [issues] alexAubin closed [issue #2385](https://github.com/YunoHost/issues/issues/2385): Use Custom Certificate Authority
[12:27:56] <ChriChri> > <@yunohostinfra:matrix.org> [yunohost] chri2 [commented](https://github.com/YunoHost/yunohost/pull/547#issuecomment-2119214570) on [issue #547](https://github.com/YunoHost/yunohost/pull/547) Fix http2 with curl: https://github.com/YunoHost/issues/issues/2396#issuecomment-2119214199

Aleks (he/him/il/lui) - do you think this is wrong? BTW: did you see this PR https://github.com/YunoHost/yunohost/pull/1843
[12:28:59] <Aleks (he/him/il/lui)> yes, i saw that PR
[12:29:44] <Aleks (he/him/il/lui)> and yes i'm not sure it makes sense to bump a 6 years old PR with a comment ...
[12:30:26] <Aleks (he/him/il/lui)> and this "sleep 2" is probably here for a reason and is no big deal honestly, i'd rather have a "sleep 2" than random failures because nginx didn't restart instantly or whatever
[12:30:30] <Aleks (he/him/il/lui)> it's not broken, don't fix it
[12:32:34] <ChriChri> > <@Alekswag:matrix.org> and yes i'm not sure it makes sense to bump a 6 years old PR with a comment ...

Yeah, probably true, but I thought it might document the context in which the sleep might get lost and warn anybody relying on it.
If the 2 sec sleep really fixes something, than it is in the wrong place in my opinion. And if we want to check this here I'd rather check for the availability of the url and wait if needed up to whatever default of seconds.
[12:33:02] <ChriChri> My guess is that ynh_systemd_action became available after that workaround.
[12:34:11] <Aleks (he/him/il/lui)> dude just ... don't overfocus on stupid details like this and get lost into it ... you wanted to improve ynh_local_curl with proper exit code and ability to check the content so just do this ...
[12:47:52] <ChriChri> > <@Alekswag:matrix.org> dude just ... don't overfocus on stupid details like this and get lost into it ... you wanted to improve ynh_local_curl with proper exit code and ability to check the content so just do this ...

Is it acceptable to add additional optional parameters like `-l --line-match: check answer for a regex to return true`, `-p --port: port to connect to`, `-n --no_https: connect using http instead of https`?
All this wouldn't change the expected behaviour.
[12:48:42] <ChriChri> In my use case I need to connect to couchdb on port 5984 using http...
[12:54:40] <Aleks (he/him/il/lui)> line-match makes sense, yes, but port and no_https, it feels more like we should be able to provide an absolute URL rather than the URI
[12:54:55] <Aleks (he/him/il/lui)> and we could just detect this by checking if the provided url/uri starts with `/`
[13:00:29] <ChriChri> something like `if parameter "page_uri" starts with 'https?://' interpreted it as an url` ? →
```
# | arg: page_uri - Path (relative to `$path_url`) of the page where POST data will be sent. If it starts with 'protocol://' its interpreted as an URL
```
[13:01:10] <ChriChri> another one: `# | arg: -H --header: add a header to the request (can be used multiple times)` → https://github.com/YunoHost/issues/issues/1696
[13:03:25] <Aleks (he/him/il/lui)> i guess, but just do one thing at a time ...
[13:05:33] <ChriChri> > <@Alekswag:matrix.org> i guess, but just do one thing at a time ...

Yes, on it: planning ahead what needs to be filled and tested :).
[13:16:43] <ChriChri> How about this version then:
```
# Curl abstraction to help with POST requests to local pages (such as installation forms)
#
# usage: ynh_local_curl "page" "key1=value1" "key2=value2" ...
# | arg: -l --line-match: check answer for a regex to return true
# | arg: -H --header: add a header to the request (can be used multiple times)
# | arg: -n --no_wait: don't wait 2 seconds *eg*
# | arg: -d --data: data to be posted if not provided as optional "key=value" pairs.
# | arg: page - either the PAGE part in 'https://$domain/$path/PAGE' or an URL like 'http://doma.in/path/file.ext'
# | arg: key1=value1 - (Optional) POST key and corresponding value
# | arg: key2=value2 - (Optional) Another POST key and corresponding value
# | arg: ... - (Optional) More POST keys and values
#
# example: ynh_local_curl "/install.php?installButton" "foo=$var1" "bar=$var2"
# → will open a POST request to "https://$domain/$path/install.php?installButton" posting "foo=$var1" and "bar=$var2"
# example: ynh_local_curl --header "Accept: application/json" -H "Content-Type: application/json" \
# --data "{\"members\":{\"names\": [\"${app}\"],\"roles\": [\"editor\"]}}" -l '"ok":true' \
# "http://localhost:5984/"
# → will open a POST request to "http://localhost:5984/" adding headers with "Accept: application/json"
# and "Content-Type: application/json" sending the data from the "--data" argument. ynh_local_curl will
# return with an error if the servers response does not match the extended regex '"ok":true'.
#
# For multiple calls, cookies are persisted between each call for the same app.
#
# `$domain` and `$path_url` need to be defined externally if the first form for the 'page' argument is used.
#
```
[13:17:23] <selfhoster1312> i don't undertand some things ConfigPanel and /etc/yunohost/settings.yml ... isn't the yml version supposed to only store what is non-default settings in "diff" mode? i have some settings in there with the default value...
also what happens when in the third sublevel some settings have the same key... do we just "pray" it does not happen?
[13:19:59] <ChriChri> PUT?
[13:19:59] <ChriChri> > <@chrichri:librem.one> How about this version then:
> ```
> # Curl abstraction to help with POST requests to local pages (such as installation forms)
> #
> # usage: ynh_local_curl "page" "key1=value1" "key2=value2" ...
> # | arg: -l --line-match: check answer for a regex to return true
> # | arg: -H --header: add a header to the request (can be used multiple times)
> # | arg: -n --no_wait: don't wait 2 seconds *eg*
> # | arg: -d --data: data to be posted if not provided as optional "key=value" pairs.
> # | arg: page - either the PAGE part in 'https://$domain/$path/PAGE' or an URL like 'http://doma.in/path/file.ext'
> # | arg: key1=value1 - (Optional) POST key and corresponding value
> # | arg: key2=value2 - (Optional) Another POST key and corresponding value
> # | arg: ... - (Optional) More POST keys and values
> #
> # example: ynh_local_curl "/install.php?installButton" "foo=$var1" "bar=$var2"
> # → will open a POST request to "https://$domain/$path/install.php?installButton" posting "foo=$var1" and "bar=$var2"
> # example: ynh_local_curl --header "Accept: application/json" -H "Content-Type: application/json" \
> # --data "{\"members\":{\"names\": [\"${app}\"],\"roles\": [\"editor\"]}}" -l '"ok":true' \
> # "http://localhost:5984/"
> # → will open a POST request to "http://localhost:5984/" adding headers with "Accept: application/json"
> # and "Content-Type: application/json" sending the data from the "--data" argument. ynh_local_curl will
> # return with an error if the servers response does not match the extended regex '"ok":true'.
> #
> # For multiple calls, cookies are persisted between each call for the same app.
> #
> # `$domain` and `$path_url` need to be defined externally if the first form for the 'page' argument is used.
> #
> ```

Uh, no PUT, yet? I mixed something up...
[13:20:03] <selfhoster1312> ChriChri, i haven't followed the discussion but that looks good... just this : `don't wait 2 seconds *eg*` is not very clear
[13:20:58] <selfhoster1312> ChriChri, maybe -X like curl to set the verb?
[13:21:13] <Yunohost Git/Infra notifications> [yunohost] alexAubin edited a [comment](https://github.com/YunoHost/yunohost/pull/1838#discussion_r1605799369) on pull request #1838 Add redis, mongodb and ruby helpers to the core: suggestion if dpkg --compare-versions {YNH_APP_PACKAGING_FORMAT:-0} lt 2 then build_ruby_dependencies="libjemalloc...
[13:23:42] <ChriChri> Another idea could be: leave the old command alone and introduce two new ones like `ynh_local_put` and `ynh_local_post`?
[13:25:03] <Yunohost Git/Infra notifications> [issues] chri2 [commented](https://github.com/YunoHost/issues/issues/2396#issuecomment-2119238124) on [issue #2396](https://github.com/YunoHost/issues/issues/2396) improve ynh_local_curl: # Curl abstraction to help with POST requests to local pages (such as installation forms) # # usage: ynh_local_curl ...
[13:25:58] <selfhoster1312> ChriChri, why introduce different commands when curl has simple -X/--request flag for this? it makes the docs longer to read :P
[13:26:36] <selfhoster1312> (also it should be `ynh_local_curl_put` and not `ynh_local_put` because there are other things than the web out there)
[13:34:09] <ChriChri> > <selfhoster1312> ChriChri, i haven't followed the discussion but that looks good... just this : `don't wait 2 seconds *eg*` is not very clear

Uh, that is more or less a joke ;-) - you can find the background in the issue...
[13:35:37] <Yunohost Git/Infra notifications> [issues] chri2 [commented](https://github.com/YunoHost/issues/issues/2396#issuecomment-2119238124) on [issue #2396](https://github.com/YunoHost/issues/issues/2396) improve ynh_local_curl: # Curl abstraction to help with POST requests to local pages (such as installation forms) # # usage: ynh_local_curl ...
[13:36:25] <selfhoster1312> ChriChri, i haven't tried but this looks reasonable: https://serverfault.com/questions/1032605/how-to-detect-when-nginx-reload-is-done-transferring-traffic
you could try on your own server and if it works it would allow to remove the sleep
[13:37:03] <Aleks (he/him/il/lui)> or we can just keep the sleep and work on meaningful stuff instead ...
[13:38:35] <selfhoster1312> but ChriChri anyway the docs should be clear and not "more or less a joke" for people who know an issue... i don't understand what the "eg" means there...
[13:44:19] <ChriChri> > <selfhoster1312> ChriChri, i haven't tried but this looks reasonable: https://serverfault.com/questions/1032605/how-to-detect-when-nginx-reload-is-done-transferring-traffic
> you could try on your own server and if it works it would allow to remove the sleep

Thanks, that's interesting - I briefly looked for something like this and didn't come across it. But Aleks (he/him/il/lui) is right: important stuff first. IMHO the waiting needs to go into that part of integration where the event happens we're supposed to wait for.
And yes, I'll clean out the joke when done testing without the 2 seconds waiting (five kids are keeping me busy so I'm a bit alergic to meaningless waiting :rotfl:).
[13:47:24] <Yunohost Git/Infra notifications> [yunohost] alexAubin edited [pull request #1840](https://github.com/YunoHost/yunohost/pull/1840): Add default_character_set options to mysql helpers
[13:47:49] <Yunohost Git/Infra notifications> [yunohost] alexAubin closed [pull request #1839](https://github.com/YunoHost/yunohost/pull/1839): Update ynh_mysql_dump_db in mysql helper
[13:47:49] <Yunohost Git/Infra notifications> [yunohost] alexAubin [commented](https://github.com/YunoHost/yunohost/pull/1839#issuecomment-2119245089) on [issue #1839](https://github.com/YunoHost/yunohost/pull/1839) Update ynh_mysql_dump_db in mysql helper: Integrated in https://github.com/YunoHost/yunohost/pull/1840
[13:50:45] <Yunohost Git/Infra notifications> [yunohost] alexAubin edited [pull request #1838](https://github.com/YunoHost/yunohost/pull/1838): Add redis and ruby helpers
[13:50:56] <Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 14 commits to dev ([48c673478d3b...f6d38bebee0e](https://github.com/YunoHost/yunohost/compare/48c673478d3b...f6d38bebee0e))
[13:50:57] <Yunohost Git/Infra notifications> [yunohost] alexAubin merged [pull request #1838](https://github.com/YunoHost/yunohost/pull/1838): Add redis and ruby helpers
[13:51:03] <Yunohost Git/Infra notifications> [yunohost/dev] Update helpers/ruby - Alexandre Aubin
[13:51:34] <Yunohost Git/Infra notifications> [yunohost] alexAubin created new branch mongodb-helper
[13:51:35] <Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 1 commit to mongodb-helper: Revert "Delete helpers/mongodb, to be readded in another PR" This reverts commit 9b7b265cbf4dd4e8e11a90b64f828dc1a4d09a... ([437f21ed](https://github.com/YunoHost/yunohost/commit/437f21ed5a616786e8ffb43a5b73b02cbda715b9))
[13:53:01] <Yunohost Git/Infra notifications> [yunohost] alexAubin opened [pull request #1844](https://github.com/YunoHost/yunohost/pull/1844): Revert "Delete helpers/mongodb, to be readded in another PR"
[13:53:09] <Yunohost Git/Infra notifications> [yunohost] alexAubin edited [pull request #1844](https://github.com/YunoHost/yunohost/pull/1844): Add mongodb helpers
[14:00:27] <Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/11.2.11.3+202405191400 for bullseye/unstable/all ...
[14:01:57] <Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/11.2.11.3+202405191400 for bullseye/unstable/all.
[15:03:00] <Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1296616539](https://gitlab.com/YunoHost/yunohost/-/pipelines/1296616539) failed on branch dev, mongodb-helper
[15:09:37] <Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1296616652](https://gitlab.com/YunoHost/yunohost/-/pipelines/1296616652) failed on branch mongodb-helper
[18:47:44] <Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 68 commits to bookworm ([dc5c43255b4e...42d2f789cf99](https://github.com/YunoHost/yunohost/compare/dc5c43255b4e...42d2f789cf99))
[18:47:49] <Yunohost Git/Infra notifications> [yunohost/bookworm] Delete helpers/mongodb, to be readded in another PR - Alexandre Aubin
[18:49:48] <Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1296721579](https://gitlab.com/YunoHost/yunohost/-/pipelines/1296721579) failed on branch bookworm
[18:52:02] <Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 1 commit to dev: Update log.py: fix fstring with no arg ([0a5dd1b0](https://github.com/YunoHost/yunohost/commit/0a5dd1b09976f2a3755c7fadb7df93db855a783e))
[18:52:58] <Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 2 commits to bookworm ([42d2f789cf99...c02933cce439](https://github.com/YunoHost/yunohost/compare/42d2f789cf99...c02933cce439))
[18:53:04] <Yunohost Git/Infra notifications> [yunohost/bookworm] Update log.py: fix fstring with no arg - Alexandre Aubin
[18:56:56] <Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1296723381](https://gitlab.com/YunoHost/yunohost/-/pipelines/1296723381) failed on branch bookworm
[19:00:33] <Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/11.2.11.3+202405191900 for bullseye/unstable/all ...
[19:01:24] <Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 1 commit to dev: helpers/ruby: fix unbound variable ([14a84453](https://github.com/YunoHost/yunohost/commit/14a8445375013716f2cdc1e16ad588a76fa49761))
[19:01:50] <Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 2 commits to bookworm ([c02933cce439...d4ef6461f38d](https://github.com/YunoHost/yunohost/compare/c02933cce439...d4ef6461f38d))
[19:01:59] <Yunohost Git/Infra notifications> [yunohost/bookworm] helpers/ruby: fix unbound variable - Alexandre Aubin
[19:02:05] <Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/11.2.11.3+202405191900 for bullseye/unstable/all.
[19:02:13] <Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/12.0.0+202405191900 for bookworm/unstable/all ...
[19:03:40] <Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/12.0.0+202405191900 for bookworm/unstable/all.
[19:04:12] <Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1296723400](https://gitlab.com/YunoHost/yunohost/-/pipelines/1296723400) failed on branch dev, bookworm
[19:05:25] <Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1296728983](https://gitlab.com/YunoHost/yunohost/-/pipelines/1296728983) failed on branch bookworm
[20:27:30] <Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1296729004](https://gitlab.com/YunoHost/yunohost/-/pipelines/1296729004) failed on branch dev, bookworm
[21:36:57] <Yunohost Git/Infra notifications> [yunohost-admin/dev] Translated using Weblate (Galician) Currently translated at 100.0% (557 of 557 strings) Translation: YunoHost/admin Tr... - José M
[21:36:57] <Yunohost Git/Infra notifications> [yunohost-admin] alexAubin pushed 3 commits to dev ([4aec50c0b9fe...5685bf993445](https://github.com/YunoHost/yunohost-admin/compare/4aec50c0b9fe...5685bf993445))
[21:36:57] <Yunohost Git/Infra notifications> [yunohost-admin] alexAubin merged [pull request #571](https://github.com/YunoHost/yunohost-admin/pull/571): Translations update from Weblate
[21:39:04] <Yunohost Git/Infra notifications> [moulinette] alexAubin pushed 4 commits to dev ([95503ca2e6de...65d694280c4f](https://github.com/YunoHost/moulinette/compare/95503ca2e6de...65d694280c4f))
[21:39:05] <Yunohost Git/Infra notifications> [moulinette] alexAubin merged [pull request #357](https://github.com/YunoHost/moulinette/pull/357): Translations update from Weblate
[21:39:19] <Yunohost Git/Infra notifications> [moulinette/dev] Translated using Weblate (Spanish) Currently translated at 97.7% (44 of 45 strings) Translation: YunoHost/moulinette T... - rosbeef andino
[21:45:15] <Yunohost Git/Infra notifications> 🏗️ Starting build for moulinette/11.2+202405192145 for bullseye/unstable/all ...
[21:46:29] <Yunohost Git/Infra notifications> ✔️ Completed build for moulinette/11.2+202405192145 for bullseye/unstable/all.
[21:46:48] <Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost-admin/11.2.5+202405192145 for bullseye/unstable/all ...
[21:50:06] <Yunohost Git/Infra notifications> ✔️ Completed build for yunohost-admin/11.2.5+202405192145 for bullseye/unstable/all.
[21:50:09] <Yunohost Git/Infra notifications> [yunohost] selfhoster1312 opened [pull request #1845](https://github.com/YunoHost/yunohost/pull/1845): Remove useless dict lookup, as the keys are always the same two
[21:52:36] <selfhoster1312> dict + list = TypeError
but dict += list == dict + list.keys()
thanks python <3
[21:53:19] <selfhoster1312> reminds me of this hilarious presentation about javascript typing: https://www.destroyallsoftware.com/talks/wat :D
[21:58:06] <Yunohost Git/Infra notifications> [moulinette] OniriCorpe pushed 1 commit to debian/11.2.1: chores: update changelog for 11.2.1 & add make_changelog script ([47930cc1](https://github.com/YunoHost/moulinette/commit/47930cc1a65172afaf00e861758d2a79b360fa97))
[21:58:07] <Yunohost Git/Infra notifications> [moulinette] OniriCorpe created new tag debian/11.2.1
[21:58:09] <Yunohost Git/Infra notifications> [moulinette] OniriCorpe pushed 1 commit to dev: chores: update changelog for 11.2.1 & add make_changelog script ([47930cc1](https://github.com/YunoHost/moulinette/commit/47930cc1a65172afaf00e861758d2a79b360fa97))
[22:00:16] <Yunohost Git/Infra notifications> 🏗️ Starting build for moulinette/11.2.1+202405192200 for bullseye/unstable/all ...
[22:01:33] <Yunohost Git/Infra notifications> ✔️ Completed build for moulinette/11.2.1+202405192200 for bullseye/unstable/all.
[22:08:04] <Yunohost Git/Infra notifications> [yunohost] OniriCorpe created new tag debian/11.2.12
[22:08:04] <Yunohost Git/Infra notifications> [yunohost] OniriCorpe pushed 1 commit to debian/11.2.12: Update 11.2.12 ([71dabfea](https://github.com/YunoHost/yunohost/commit/71dabfea6c1d1ae544fe1b86930a6c3ccc5eb27a))
[22:08:09] <Yunohost Git/Infra notifications> [yunohost] OniriCorpe pushed 1 commit to dev: Update 11.2.12 ([71dabfea](https://github.com/YunoHost/yunohost/commit/71dabfea6c1d1ae544fe1b86930a6c3ccc5eb27a))
[22:09:57] <Yunohost Git/Infra notifications> [doc.yunohost.org] yunohost-bot pushed 1 commit to master: docs: update make-a-release ([5baedfe9](https://github.com/YunoHost/doc.yunohost.org/commit/5baedfe9de3b6742e03032ba588b1c9fecda0d05))
[22:15:19] <Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/11.2.12+202405192215 for bullseye/unstable/all ...
[22:16:53] <Yunohost Git/Infra notifications> [yunohost-admin] OniriCorpe created new tag debian/11.2.6
[22:16:53] <Yunohost Git/Infra notifications> [yunohost-admin] OniriCorpe pushed 2 commits to debian/11.2.6 ([8175590e8eed^...31fca33df960](https://github.com/YunoHost/yunohost-admin/compare/8175590e8eed^...31fca33df960))
[22:16:57] <Yunohost Git/Infra notifications> [yunohost-admin/debian/11.2.6] chores: add make_changelog script - OniriCorpe
[22:17:04] <Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/11.2.12+202405192215 for bullseye/unstable/all.
[22:20:00] <Yunohost Git/Infra notifications> [yunohost-admin] OniriCorpe deleted tag debian/11.2.6
[22:21:06] <Yunohost Git/Infra notifications> [yunohost-admin] OniriCorpe created new tag debian/11.2.6
[22:21:07] <Yunohost Git/Infra notifications> [yunohost-admin] OniriCorpe pushed 2 commits to dev ([5685bf993445...ad7c927aa191](https://github.com/YunoHost/yunohost-admin/compare/5685bf993445...ad7c927aa191))
[22:21:07] <Yunohost Git/Infra notifications> [yunohost-admin] OniriCorpe pushed 2 commits to debian/11.2.6 ([653689e8ae61^...ad7c927aa191](https://github.com/YunoHost/yunohost-admin/compare/653689e8ae61^...ad7c927aa191))
[22:25:22] <lapineige> (And that's off topic ?)
[22:28:33] <Yunohost Git/Infra notifications> [SSOwat] OniriCorpe created new tag debian/11.2.1
[22:28:33] <Yunohost Git/Infra notifications> [SSOwat/dev] chores: add make_changelog script - OniriCorpe
[22:28:33] <Yunohost Git/Infra notifications> [SSOwat] OniriCorpe pushed 2 commits to dev ([9f0ca467170e...542a87f78039](https://github.com/YunoHost/SSOwat/compare/9f0ca467170e...542a87f78039))
[22:28:33] <Yunohost Git/Infra notifications> [SSOwat] OniriCorpe pushed 2 commits to debian/11.2.1 ([9c8482f2f28c^...542a87f78039](https://github.com/YunoHost/SSOwat/compare/9c8482f2f28c^...542a87f78039))
[22:32:22] <Yunohost Git/Infra notifications> [moulinette] OniriCorpe pushed 1 commit to dev: chores: update changelog for 11.2.1 & add make_changelog script ([3ed8fb13](https://github.com/YunoHost/moulinette/commit/3ed8fb13638b8008a8189d29d073521daa9a85d0))
[22:32:22] <Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost-admin/11.2.6+202405192230 for bullseye/unstable/all ...
[22:32:22] <Yunohost Git/Infra notifications> [moulinette] OniriCorpe pushed 1 commit to debian/11.2.1: chores: update changelog for 11.2.1 & add make_changelog script ([3ed8fb13](https://github.com/YunoHost/moulinette/commit/3ed8fb13638b8008a8189d29d073521daa9a85d0))
[22:32:22] <Yunohost Git/Infra notifications> [moulinette] OniriCorpe deleted tag debian/11.2.1
[22:32:22] <Yunohost Git/Infra notifications> ❌ Failed build for SSOwat/11.2.1+202405192230 for bullseye/unstable/all ?!
[22:32:23] <Yunohost Git/Infra notifications> 🏗️ Starting build for SSOwat/11.2.1+202405192230 for bullseye/unstable/all ...
[22:32:51] <Yunohost Git/Infra notifications> [moulinette] OniriCorpe deleted tag debian/11.2.1
[22:33:33] <Yunohost Git/Infra notifications> [moulinette] OniriCorpe pushed 1 commit to dev: chores: update changelog for 11.2.1 & add make_changelog script ([497ea8a4](https://github.com/YunoHost/moulinette/commit/497ea8a4af1c0b486837e72b5bf757848f8e1854))
[22:33:39] <Yunohost Git/Infra notifications> [moulinette] OniriCorpe created new tag debian/11.2.1
[22:33:59] <Yunohost Git/Infra notifications> ✔️ Completed build for yunohost-admin/11.2.6+202405192230 for bullseye/unstable/all.
[22:36:46] <Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/11.2.12 for bullseye/stable/all ...
[22:37:46] <Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/11.2.12 for bullseye/stable/all.
[22:37:52] <Yunohost Git/Infra notifications> 🏗️ Starting build for moulinette/11.2.1 for bullseye/stable/all ...
[22:38:49] <Yunohost Git/Infra notifications> ✔️ Completed build for moulinette/11.2.1 for bullseye/stable/all.
[22:38:55] <Yunohost Git/Infra notifications> 🏗️ Starting build for SSOwat/11.2.1 for bullseye/stable/all ...
[22:39:00] <Yunohost Git/Infra notifications> ❌ Failed build for SSOwat/11.2.1 for bullseye/stable/all ?!
[22:39:29] <Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost-admin/11.2.6 for bullseye/stable/all ...
[22:42:43] <Yunohost Git/Infra notifications> ✔️ Completed build for yunohost-admin/11.2.6 for bullseye/stable/all.
[22:44:04] <Yunohost Git/Infra notifications> [SSOwat] alexAubin created new tag debian/11.2.1.1
[22:44:05] <Yunohost Git/Infra notifications> [SSOwat] alexAubin pushed 1 commit to dev: Fix changelog upper/lowercase for 11.2.1.1 ([4826e7aa](https://github.com/YunoHost/SSOwat/commit/4826e7aa09ac3c4631b7251c8359698f6eed9531))
[22:44:43] <Yunohost Git/Infra notifications> 🏗️ Starting build for ssowat/11.2.1.1 for bullseye/stable/all ...
[22:45:15] <Yunohost Git/Infra notifications> 🏗️ Starting build for moulinette/11.2.1+202405192245 for bullseye/unstable/all ...
[22:45:56] <Yunohost Git/Infra notifications> ✔️ Completed build for ssowat/11.2.1.1 for bullseye/stable/all.
[22:46:53] <Yunohost Git/Infra notifications> ✔️ Completed build for moulinette/11.2.1+202405192245 for bullseye/unstable/all.
[22:47:01] <Émy - OniriCorpe> ```text
Les paquets suivants seront mis à jour :
moulinette ssowat yunohost yunohost-admin
```
[22:47:02] <Yunohost Git/Infra notifications> 🏗️ Starting build for ssowat/11.2.1.1+202405192245 for bullseye/unstable/all ...
[22:47:02] <Émy - OniriCorpe> 🎉
[22:48:03] <Yunohost Git/Infra notifications> ✔️ Completed build for ssowat/11.2.1.1+202405192245 for bullseye/unstable/all.
[23:46:59] <Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 1 commit to dev: doc: fix gitlab helper/resource doc automagic PR because doc structure changed ([d5e054fe](https://github.com/YunoHost/yunohost/commit/d5e054fe80002862bd5966d61b20e9fafcf0578e))
[23:48:48] <Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1296791915](https://gitlab.com/YunoHost/yunohost/-/pipelines/1296791915) failed on branch dev