[02:42:44]
<Yunohost Git/Infra notifications> [yunohost] utkarshsethi [commented](https://github.com/YunoHost/yunohost/pull/1820#issuecomment-2453732082) on [issue #1820](https://github.com/YunoHost/yunohost/pull/1820) #2128 adding issuewild: I dont know what is supposed to happen here until https://github.com/YunoHost/issues/issues/2089 becomes a priority eit...
[02:44:26]
<Yunohost Git/Infra notifications> [yunohost] utkarshsethi [commented](https://github.com/YunoHost/yunohost/pull/1820#issuecomment-2453732082) on [issue #1820](https://github.com/YunoHost/yunohost/pull/1820) #2128 adding issuewild: I dont know what is supposed to happen here until https://github.com/YunoHost/issues/issues/2089 or https://github.com/...
[02:45:03]
<Yunohost Git/Infra notifications> [yunohost] utkarshsethi [commented](https://github.com/YunoHost/yunohost/pull/1820#issuecomment-2453732082) on [issue #1820](https://github.com/YunoHost/yunohost/pull/1820) #2128 adding issuewild: I dont know what is supposed to happen here until https://github.com/YunoHost/issues/issues/2089 or https://github.com/...
[02:45:10]
<Yunohost Git/Infra notifications> [yunohost] utkarshsethi [commented](https://github.com/YunoHost/yunohost/pull/1820#issuecomment-2453733820) on [issue #1820](https://github.com/YunoHost/yunohost/pull/1820) #2128 adding issuewild: I dont know what is supposed to happen here until https://github.com/YunoHost/issues/issues/2089 or https://github.com/...
[07:00:32]
<Yunohost Git/Infra notifications> gabrielcossette forked install_script to [gabrielcossette/yunohost_install_script](https://github.com/gabrielcossette/yunohost_install_script)
[08:42:16]
<florent> Hi there,
Can anyone take a look at this PR: https://github.com/YunoHost/package_check/pull/178
It should fix an issue I have with the package checker, which erronously consider a package of mine as broken (due to the fact the checker does not consider the html `<base>` tag).
Thanks in advance
[09:26:04]
<Salamandar> imho you don't need to return the resolved_path because it can be "calculated" by the caller
[09:29:35]
<Salamandar> also i don't get why we pass base_url and path separately to curl() in the first place :D
[09:32:39]
<florent> > <@Salamandar:matrix.org> imho you don't need to return the resolved_path because it can be "calculated" by the caller
It's a bit hacky, I prefer having it called by the function at a single point.
[09:33:06]
<Salamandar> what I mean is that curl() takes both base_url and path, but it could only take 1 argument
[09:33:25]
<Salamandar> (that's not a remark on your PR, but your PR makes it more obvious)
[09:34:14]
<Salamandar> hmmm
[09:34:27]
<florent> Well, I prefer being chirurgical and only focus on the fix for that. Otherwise I tend to agree (or it may be a separate function that would concatenate both the base_url and the path)
[09:34:29]
<Salamandar> i'm not an expert on those things, but it seems like your PR won't cover all `base` cases
[09:34:30]
<Salamandar> https://developer.mozilla.org/en-US/docs/Web/HTML/Element/base
[09:34:55]
<Salamandar> https://github.com/YunoHost/package_check/pull/178/files#diff-b3fdf547c9098a5988f344c062ad1ba43089007aa4118d5bdbb840d111ea4749R240
[09:35:20]
<Salamandar> looks like you're gonna curl() `https://{domain}{base}/{asset}`
[09:35:30]
<Salamandar> but base *might* contain the domain
[09:37:25]
<florent> I thought I had covered the case, let me check.
[09:39:10]
<florent> You're right, it is not covered
[09:39:21]
<Salamandar> imho we should do `base = base or f"https://{domain}"` or something similar
[09:39:35]
<florent> It may have been when I used urljoin for concatenating the urls
[09:40:16]
<Salamandar> aaa
[09:40:18]
<florent> > <@Salamandar:matrix.org> imho we should do `base = base or f"https://{domain}"` or something similar
you mean the passed argument?
[09:40:19]
<Salamandar> i see what you mean
[09:40:22]
<Salamandar> ```
>>> from urllib.parse import urlencode, urljoin
>>> urljoin("https://a", "b")
'https://a/b'
>>> urljoin("https://a", "https://b")
'https://b'
```
[09:41:09]
<Salamandar> hmmm
[09:41:28]
<Salamandar> i'ma simplify the code a bit ; is it OK with you if i rebase/force push your branch with some changes ?
[09:44:50]
<Yunohost Git/Infra notifications> [package_check] Salamandar created new branch simplify_curl
[09:44:53]
<Yunohost Git/Infra notifications> [package_check] Salamandar pushed 1 commit to simplify_curl: curl_tests: Simplify curl() by only passing one argument, the full url ([734dd609](https://github.com/YunoHost/package_check/commit/734dd6092c3a15e35e339b2666360f32267f9ea5))
[09:44:59]
<Yunohost Git/Infra notifications> [package_check] Salamandar opened [pull request #181](https://github.com/YunoHost/package_check/pull/181): curl_tests: Simplify curl() by only passing one argument, the full url
[09:45:09]
<Yunohost Git/Infra notifications> [package_check] Salamandar merged [pull request #181](https://github.com/YunoHost/package_check/pull/181): curl_tests: Simplify curl() by only passing one argument, the full url
[09:45:10]
<Yunohost Git/Infra notifications> [package_check] Salamandar pushed 1 commit to master: curl_tests: Simplify curl() by only passing one argument, the full url ([a31421a0](https://github.com/YunoHost/package_check/commit/a31421a01ac8dae45c3d0cd639543b59484f6d8d))
[09:45:12]
<Yunohost Git/Infra notifications> [package_check] Salamandar deleted branch simplify_curl
[09:47:11]
<florent> Yeah, please do. As long as my original commit is present, as much as possible
[09:47:32]
<Salamandar> Yeah, that's the idea, at least you keep commit authorship
[09:48:17]
<florent> I don't care much about that, just that I want to be able to come back to the original idea if necessary :)
[09:48:41]
<Salamandar> Well i just simplified curl() as I explained earlier, now i'm just adapting your code :) I'm keeping the base() and urljoin() logic as original
[09:48:52]
<Salamandar> Well i just simplified curl() as I explained earlier, now i'm just adapting your code :) I'm keeping the base and urljoin() logic as original
[09:49:19]
<florent> Be sure you have this pitfall in mind:
https://github.com/YunoHost/package_check/pull/176#issuecomment-2429601078
[09:56:29]
<Salamandar> lol that's annoying
[09:57:48]
<Salamandar> although i don't get it… an asset with href="/something" should be on the root, not relative to the page url
[10:14:01]
<Salamandar> > I don't care much about that, just that I want to be able to come back to the original idea if necessary :)
done
[10:14:48]
<Salamandar> I also kept your second commit https://github.com/YunoHost/package_check/pull/178/commits/547ee0e21812b11db03fa5edc5f3ec31a62a4641 that replaces urljoin() with a concat, to discuss on it
[10:15:24]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 15 commits to 30.0.0 ([68a855d58380...3356d9d4d2a9](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/68a855d58380...3356d9d4d2a9))
[10:15:25]
<Yunohost Git/Infra notifications> [nextcloud_ynh/30.0.0] cleaning - Éric Gaspar
[10:15:25]
<Yunohost Git/Infra notifications> [nextcloud_ynh/30.0.0] cleaning - Éric Gaspar
[10:15:26]
<Yunohost Git/Infra notifications> [nextcloud_ynh/30.0.0] Merge branch testing into 30.0.0 - Éric Gaspar
[10:17:58]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 20 commits to manage-oldstable-version ([6afb7b9b41ea...dc736b76b137](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/6afb7b9b41ea...dc736b76b137))
[10:18:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh/manage-oldstable-version] cleaning - Éric Gaspar
[10:18:10]
<Yunohost Git/Infra notifications> [nextcloud_ynh/manage-oldstable-version] cleaning - Éric Gaspar
[10:18:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh/manage-oldstable-version] Merge branch testing into manage-oldstable-version - Éric Gaspar
[10:55:16]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to v2_PostgreSQL: cleaning ([62ed82cf](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/62ed82cff2ad7c06ab106d33d209a659ea4da2c9))
[10:55:16]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to v2_PostgreSQL: cleaning ([67ea5d81](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/67ea5d8100af382b64ac39db1588532b0d020eb4))
[10:55:17]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot pushed 1 commit to v2_PostgreSQL: Auto-update READMEs ([3fce63ed](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/3fce63ed265b1e4a7789a0b10c0050b89dd0c7c5))
[11:10:24]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to testing: add link to the help menu ([27814047](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/278140470a66eb6b79fc58a08f1451c46d8ce682))
[11:11:30]
<Yunohost Git/Infra notifications> [package_check] fflorent [commented](https://github.com/YunoHost/package_check/pull/178#discussion_r1827563918) on pull request #178 Fix wrong asset 404 error: What may work is using urljoin specifically here: suggestion resolved_asset_url = urljoin("https://{dom...
[11:12:37]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar created new branch remove-OnlyOffice-doc
[11:12:51]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to remove-OnlyOffice-doc: remove onlyoffice doc ([9a9083c3](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/9a9083c38cdd3977f9c2279509af7dd6de2636bf))
[11:14:29]
<Yunohost Git/Infra notifications> [package_check] fflorent [commented](https://github.com/YunoHost/package_check/pull/178#discussion_r1827567535) on pull request #178 Fix wrong asset 404 error: To be honest, I dont remember why I committed that. If I am not wrong, domain should only contain a domain as its nam...
[11:14:33]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar opened [pull request #743](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/743): remove onlyoffice doc
[11:15:12]
<florent> > <@Salamandar:matrix.org> I also kept your second commit https://github.com/YunoHost/package_check/pull/178/commits/547ee0e21812b11db03fa5edc5f3ec31a62a4641 that replaces urljoin() with a concat, to discuss on it
I answered, I would be happy to revert if the tests pass.
[12:12:06]
<Yunohost Git/Infra notifications> [nextcloud_ynh/30.0.0] add link to the help menu - Éric Gaspar
[12:12:07]
<Yunohost Git/Infra notifications> [nextcloud_ynh/30.0.0] Merge branch testing into 30.0.0 - Éric Gaspar
[12:12:07]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 2 commits to 30.0.0 ([3356d9d4d2a9...13c3217faab5](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/3356d9d4d2a9...13c3217faab5))
[12:12:41]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar merged [pull request #743](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/743): remove onlyoffice doc
[12:12:41]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to testing: remove onlyoffice doc (#743) ([6f868da5](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/6f868da54a3dd64aca2d342771e9475e6a210273))
[12:12:58]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar edited [pull request #735](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/735): Testing
[12:16:14]
<Yunohost Git/Infra notifications> [nextcloud_ynh/manage-oldstable-version] add link to the help menu - Éric Gaspar
[12:16:14]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 2 commits to manage-oldstable-version ([dc736b76b137...180000e073ac](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/dc736b76b137...180000e073ac))
[12:16:15]
<Yunohost Git/Infra notifications> [nextcloud_ynh/manage-oldstable-version] Merge branch testing into manage-oldstable-version - Éric Gaspar
[12:26:59]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 27 commits to fix-acl ([8b3220ef8ca6...11a887295128](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/8b3220ef8ca6...11a887295128))
[12:27:00]
<Yunohost Git/Infra notifications> [nextcloud_ynh/fix-acl] remove onlyoffice doc (#743) - eric_G
[12:27:00]
<Yunohost Git/Infra notifications> [nextcloud_ynh/fix-acl] Merge branch testing into fix-acl - Éric Gaspar
[12:27:00]
<Yunohost Git/Infra notifications> [nextcloud_ynh/fix-acl] add link to the help menu - Éric Gaspar
[12:30:41]
<Yunohost Git/Infra notifications> [package_check] Salamandar [commented](https://github.com/YunoHost/package_check/pull/178#discussion_r1827657358) on pull request #178 Fix wrong asset 404 error: Indeed it should be safe. Lezdoit.
[12:33:20]
<Yunohost Git/Infra notifications> [package_check] Salamandar [commented](https://github.com/YunoHost/package_check/pull/178#issuecomment-2454599343) on [issue #178](https://github.com/YunoHost/package_check/pull/178) Fix wrong asset 404 error: @alexAubin it should be good imho but if you want to check it out before merging.
[12:34:39]
<Yunohost Git/Infra notifications> [package_check] fflorent [commented](https://github.com/YunoHost/package_check/pull/178#discussion_r1827662088) on pull request #178 Fix wrong asset 404 error: I checked in run against my package, [Grist](https://github.com/YunoHost-Apps/grist_ynh), it seems to work (it passed th...
[12:58:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to testing: cleaning ([96c0aaf0](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/96c0aaf02aaccd95dc3ea79dbeeed2ae795300ab))
[12:58:09]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot pushed 1 commit to testing: Auto-update READMEs ([2a296b79](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/2a296b79fe2fdadfc5041f8c5b57529a1eb45fa4))
[12:58:32]
<Yunohost Git/Infra notifications> [nextcloud_ynh/testing] Update DESCRIPTION.md - Éric Gaspar
[12:58:32]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 2 commits to testing ([2a296b79fe2f...9015178b4045](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/2a296b79fe2f...9015178b4045))
[12:58:34]
<Yunohost Git/Infra notifications> [nextcloud_ynh/testing] Merge branch testing of https://github.com/YunoHost-Apps/nextcloud_ynh into testing - Éric Gaspar
[12:58:42]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot pushed 1 commit to testing: Auto-update READMEs ([b48322e4](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/b48322e45024ee9b681f706051938c0d8784b303))
[13:00:42]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 7 commits to v2_PostgreSQL ([3fce63ed265b...3f09926f98db](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/3fce63ed265b...3f09926f98db))
[13:00:43]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Merge branch testing of https://github.com/YunoHost-Apps/nextcloud_ynh into testing - Éric Gaspar
[13:00:43]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Update DESCRIPTION.md - Éric Gaspar
[13:00:44]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Merge branch testing into v2_PostgreSQL - Éric Gaspar
[13:01:09]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 11 commits to v2_PostgreSQL_helpers_2.1 ([00de14a0d539...20b69fa1c196](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/00de14a0d539...20b69fa1c196))
[13:01:17]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot pushed 1 commit to v2_PostgreSQL: Auto-update READMEs ([51b65ef8](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/51b65ef834a58932da1977a4dab62d21afae42fc))
[13:01:19]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 7 commits to 30.0.0 ([13c3217faab5...5d84dee9d2e4](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/13c3217faab5...5d84dee9d2e4))
[13:01:20]
<Yunohost Git/Infra notifications> [nextcloud_ynh/30.0.0] Auto-update READMEs - yunohost-bot
[13:01:20]
<Yunohost Git/Infra notifications> [nextcloud_ynh/30.0.0] Merge branch testing of https://github.com/YunoHost-Apps/nextcloud_ynh into testing - Éric Gaspar
[13:01:21]
<Yunohost Git/Infra notifications> [nextcloud_ynh/30.0.0] Merge branch testing into 30.0.0 - Éric Gaspar
[13:01:21]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot pushed 1 commit to v2_PostgreSQL_helpers_2.1: Auto-update READMEs ([6d7e92f3](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/6d7e92f3c0bb23931d291d98520a00fa185b2dee))
[13:04:23]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to 30.0.0: Update screenshot.png ([5a44bfb8](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/5a44bfb8501c9798d200eafd3ebf72cd659fe854))
[13:18:34]
<Yunohost Git/Infra notifications> [package_check] alexAubin approved [pull request #178](https://github.com/YunoHost/package_check/pull/178#pullrequestreview-2413044540) Fix wrong asset 404 error: LGTM naively but we should keep a close eye on the CI after merging
[13:25:37]
<Yunohost Git/Infra notifications> cerencotukk forked yunohost to [cerencotukk/yunohost](https://github.com/cerencotukk/yunohost)
[13:42:07]
<Yunohost Git/Infra notifications> [issues] nicofrand [commented](https://github.com/YunoHost/issues/issues/2459#issuecomment-2454749642) on [issue #2459](https://github.com/YunoHost/issues/issues/2459) [bookworm] WebUI buttons do not trigger the assigned function: I can reproduce too in the upgrade section: the button triggers a modal but the modal remains hidden due to .fade:not(....
[13:43:30]
<Yunohost Git/Infra notifications> [issues] nicofrand [commented](https://github.com/YunoHost/issues/issues/2459#issuecomment-2454749642) on [issue #2459](https://github.com/YunoHost/issues/issues/2459) [bookworm] WebUI buttons do not trigger the assigned function: I can reproduce too in the upgrade section: the button triggers a modal (to confirm the action) but the modal remains h...
[14:02:07]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to 30.0.0: cleaning ([c4850932](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/c4850932a21e9870c0c40a1eac36c31d47f064aa))
[14:03:49]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to 30.0.0: cleaning ([ccf1c4cf](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/ccf1c4cf5456bed56bd5f68f5fb4c2fa1d06c166))
[14:07:38]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to v2_PostgreSQL: cleaning ([57752134](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/577521347e57f2f6f94f3f85e98fca7c4e1af156))
[14:08:16]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to v2_PostgreSQL: Update install ([f713adf5](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/f713adf5a80f0813fd7c3352f5c8571afdf57a81))
[14:28:48]
<Yunohost Git/Infra notifications> [package_check] fflorent [commented](https://github.com/YunoHost/package_check/pull/178#issuecomment-2454863309) on [issue #178](https://github.com/YunoHost/package_check/pull/178) Fix wrong asset 404 error: @alexAubin @Salamandar So the plan is that we merge this PR and in case of regression we revert? I am fine with that. F...
[14:29:13]
<Yunohost Git/Infra notifications> [package_check] fflorent [commented](https://github.com/YunoHost/package_check/pull/178#issuecomment-2454863309) on [issue #178](https://github.com/YunoHost/package_check/pull/178) Fix wrong asset 404 error: @alexAubin @Salamandar So the plan is that we merge this PR and in case of regression we revert? I agree. FWIW: I dont...
[14:29:56]
<Yunohost Git/Infra notifications> [package_check] Salamandar [commented](https://github.com/YunoHost/package_check/pull/178#issuecomment-2454866235) on [issue #178](https://github.com/YunoHost/package_check/pull/178) Fix wrong asset 404 error: Yeah, im just fixing some stuff that is already breaking the CI, then once im sure it works, lets merge this. I dont...
[14:34:32]
<Yunohost Git/Infra notifications> [package_check] fflorent [commented](https://github.com/YunoHost/package_check/pull/178#issuecomment-2454877637) on [issue #178](https://github.com/YunoHost/package_check/pull/178) Fix wrong asset 404 error: Sounds wise, indeed
[15:26:40]
<Yunohost Git/Infra notifications> [apps] yunohost-bot created new branch add-to-wishlist-openwisp
[15:26:40]
<Yunohost Git/Infra notifications> [apps] yunohost-bot pushed 1 commit to add-to-wishlist-openwisp: Add Openwisp to wishlist ([03576816](https://github.com/YunoHost/apps/commit/03576816a09012bb91cc365cff4e80f9ef3c5cef))
[15:26:40]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2681](https://github.com/YunoHost/apps/pull/2681): Add Openwisp to wishlist
[15:26:41]
<Yunohost Git/Infra notifications> [apps] yunohost-bot labeled Wishlist on [pull request #2681](https://github.com/YunoHost/apps/pull/2681): Add Openwisp to wishlist
[15:35:13]
<Yunohost Git/Infra notifications> [apps] alexAubin [commented](https://github.com/YunoHost/apps/pull/2681#issuecomment-2455032200) on [issue #2681](https://github.com/YunoHost/apps/pull/2681) Add Openwisp to wishlist: ... Feels super technical and out of scope of YunoHost ...
[16:03:08]
<Yunohost Git/Infra notifications> [apps] ericgaspar closed [pull request #2681](https://github.com/YunoHost/apps/pull/2681): Add Openwisp to wishlist
[16:03:09]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch add-to-wishlist-openwisp
[16:24:29]
<Yunohost Git/Infra notifications> [CustomPiOS] Lab-8916100448256 pushed 1 commit to yunohost: update clic install ([0f780e2a](https://github.com/YunoHost/CustomPiOS/commit/0f780e2a75f08c115956306579055d7f2a4a08c5))
[16:26:10]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to manage-oldstable-version: add some context ([5e88693d](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/5e88693d4324288a3e5eafe5f387ce02997baaa6))
[16:26:23]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 7 commits to manage-oldstable-version ([5e88693d4324...c8268299863b](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/5e88693d4324...c8268299863b))
[16:26:24]
<Yunohost Git/Infra notifications> [nextcloud_ynh/manage-oldstable-version] Merge branch testing of https://github.com/YunoHost-Apps/nextcloud_ynh into testing - Éric Gaspar
[16:26:24]
<Yunohost Git/Infra notifications> [nextcloud_ynh/manage-oldstable-version] Auto-update READMEs - yunohost-bot
[16:26:25]
<Yunohost Git/Infra notifications> [nextcloud_ynh/manage-oldstable-version] Merge branch testing into manage-oldstable-version - Éric Gaspar
[16:27:19]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar approved [pull request #714](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/714#pullrequestreview-2413535633) Manage oldstable version
[16:37:40]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to manage-oldstable-version: Update config_panel.toml ([a67fb028](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/a67fb0287604aa7852b5e3166b4fb6f5b8bed42f))
[16:40:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to manage-oldstable-version: Update config_panel.toml ([581764a5](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/581764a542f22f9db8041b88bdfc171c5227db56))
[16:40:57]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/714#issuecomment-2455193261) on [issue #714](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/714) Manage oldstable version: testme
[16:45:10]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar created new branch simplify-install-options
[16:46:37]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to simplify-install-options: cleaning options ([0c8f42cc](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/0c8f42ccedb82d6cfc3ab6fe526f66a59be19f18))
[16:47:09]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/744#issuecomment-2455206736) on [issue #744](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/744) cleaning options: testme
[16:47:10]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar opened [pull request #744](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/744): cleaning options
[16:47:14]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/744#issuecomment-2455207065) on [issue #744](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/744) cleaning options: :book: :worm:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/3708/badge)...
[16:50:38]
<Yunohost Git/Infra notifications> [nextcloud_ynh] alexAubin edited [pull request #744](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/744): Removing enable address book and enable notify push from install questions, having them in the config panel should be enough, KISS / Less is more etc
[16:50:39]
<Yunohost Git/Infra notifications> [nextcloud_ynh] alexAubin approved [pull request #744](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/744#pullrequestreview-2413594823) Removing enable address book and enable notify push from install questions, having them in the config panel should be enough, KISS / Less is more etc
[17:02:01]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar merged [pull request #744](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/744): Removing enable address book and enable notify push from install questions, having them in the config panel should be enough, KISS / Less is more etc
[17:02:02]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to testing: cleaning options (#744) ([6fdae340](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/6fdae3408f3d801f085eb1e6c17a2fdffd346875))
[17:02:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar deleted branch simplify-install-options
[17:02:20]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar edited [pull request #735](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/735): Testing
[17:02:50]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 2 commits to 30.0.0 ([ccf1c4cf5456...d0fb6d946519](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/ccf1c4cf5456...d0fb6d946519))
[17:02:51]
<Yunohost Git/Infra notifications> [nextcloud_ynh/30.0.0] cleaning options (#744) - eric_G
[17:02:54]
<Yunohost Git/Infra notifications> [nextcloud_ynh/30.0.0] Merge branch testing into 30.0.0 - Éric Gaspar
[17:04:15]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 3 commits to v2_PostgreSQL ([f713adf5a80f...4dcc964d3140](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/f713adf5a80f...4dcc964d3140))
[17:04:15]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Auto-update READMEs - yunohost-bot
[17:04:19]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Merge branch testing into v2_PostgreSQL - Éric Gaspar
[17:04:19]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] cleaning options (#744) - eric_G
[17:07:55]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar edited [pull request #735](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/735): Testing
[18:03:15]
<Yunohost Git/Infra notifications> [webhooks] alexAubin pushed 1 commit to master: Delete obsolete to_room.py ([87b49a4e](https://github.com/YunoHost/webhooks/commit/87b49a4e87596a1c72077a77cb658c45ea5354df))
[18:08:54]
<Yunohost Git/Infra notifications> test from aleks
[18:09:00]
<Yunohost Git/Infra notifications> test from aleks
[18:09:08]
<Yunohost Git/Infra notifications> test from aleks
[18:09:18]
<Aleks (he/him/il/lui)> 🤔
[18:19:48]
<Yunohost Git/Infra notifications> test from aleks
[18:22:43]
<Yunohost Git/Infra notifications> test from aleks
[18:22:51]
<Aleks (he/him/il/lui)> mais bordel de cul
[18:22:59]
<Aleks (he/him/il/lui)> are cli argument a joke to you
[18:38:16]
<Yunohost Git/Infra notifications> test from aleks
[18:47:27]
<Yunohost Git/Infra notifications> alexAubin archived repository monitor-ynh: A small infra monitoring tool https://github.com/YunoHost/monitor-ynh
[18:47:46]
<Yunohost Git/Infra notifications> alexAubin archived repository rspamd: Rapid spam filtering system. https://github.com/YunoHost/rspamd
[18:47:57]
<Yunohost Git/Infra notifications> alexAubin archived repository clinette: Command line client for DynDns admin operation https://github.com/YunoHost/clinette
[18:53:55]
<Yunohost Git/Infra notifications> [webhooks] alexAubin pushed 1 commit to master: Fixes after redeploy on new server, using room alias doesnt work so we have to hardcode the stupid room id ([975e03de](https://github.com/YunoHost/webhooks/commit/975e03dee3d2ebcb483d4b1753456ce59a5de761))
[18:54:12]
<Yunohost Git/Infra notifications> [webhooks] github-actions[bot] opened [pull request #7](https://github.com/YunoHost/webhooks/pull/7): Format Python code with Black
[18:54:15]
<Yunohost Git/Infra notifications> [webhooks] github-actions[bot] pushed 1 commit to actions/black: :art: Format Python code with Black ([11dbf6ca](https://github.com/YunoHost/webhooks/commit/11dbf6cae184c7dbbe9f15b23493336fa1f339ce))
[18:54:15]
<Yunohost Git/Infra notifications> [webhooks] github-actions[bot] created new branch actions/black
[19:37:17]
<Salamandar> @Alekswag:matrix.org is https://github.com/YunoHost/vinaigrette/blob/master/scripts/ynh-custom-builds still useful ?
[20:01:22]
<Aleks (he/him/il/lui)> maybe not but i would still keep it in the unlikely case that we need to rebuild something for bullseye, or maybe for trixie
[20:01:49]
<Aleks (he/him/il/lui)> (but what should be preferred imho is to take the .deb from an external source and add it to our repo but that's not always possible)
[20:37:45]
<Salamandar> alright
[20:37:54]
<Salamandar> tbh i have a hard time understanding the code of vinaigrette
[20:50:22]
<Aleks (he/him/il/lui)> i don't know what you don't understand, it's all straightforward
[20:54:44]
<Salamandar> lulz
[21:08:52]
<Yunohost Git/Infra notifications> [CustomPiOS] Lab-8916100448256 pushed 1 commit to yunohost: updates for ynh 12 ([7a7fb5fb](https://github.com/YunoHost/CustomPiOS/commit/7a7fb5fbc2a3f4eda36b4977f5602aae2250273e))
[21:25:44]
<Yunohost Git/Infra notifications> [vinaigrette] Salamandar pushed 1 commit to trixie: Add support for trixie builds ([5c5c8095](https://github.com/YunoHost/vinaigrette/commit/5c5c8095c8ed413fafd84676984a3c19ab42590d))
[21:25:45]
<Yunohost Git/Infra notifications> [vinaigrette] Salamandar created new branch trixie
[21:25:52]
<Yunohost Git/Infra notifications> [vinaigrette] Salamandar opened [pull request #9](https://github.com/YunoHost/vinaigrette/pull/9): Add support for trixie builds
[21:26:48]
<Yunohost Git/Infra notifications> [dynette] zamentur created new branch enh-include
[21:26:48]
<Yunohost Git/Infra notifications> [dynette] zamentur pushed 1 commit to enh-include: [enh] Avoid to generate tons of lines to grant permissions ([bb5d3df3](https://github.com/YunoHost/dynette/commit/bb5d3df3e666e0262803858e8dfa4624c704dd1e))
[21:28:03]
<Yunohost Git/Infra notifications> [dynette] zamentur opened [pull request #18](https://github.com/YunoHost/dynette/pull/18): [enh] Avoid to generate tons of lines to grant permissions
[21:29:57]
<Yunohost Git/Infra notifications> [dynette] zamentur pushed 1 commit to enh-include: [enh] Include keys file by file ([918bf56f](https://github.com/YunoHost/dynette/commit/918bf56fb0641a60507eaef05844d8385760c220))
[21:35:49]
<Yunohost Git/Infra notifications> [dynette] zamentur [commented](https://github.com/YunoHost/dynette/pull/18#discussion_r1828421001) on pull request #18 [enh] Avoid to generate tons of lines to grant permissions: See https://bind9.readthedocs.io/en/v9.18.31/reference.html#namedconf-statement-update-policy If i understand well, all...
[21:42:52]
<Yunohost Git/Infra notifications> [yunohost] tituspijean pushed 1 commit to enh-san: Support domains in cert_alternate_names hook in addition to implied subdomain of the apps domain ([a11e85ac](https://github.com/YunoHost/yunohost/commit/a11e85acd11f4285498fdb9f6a80c6187c34714a))
[21:42:52]
<Yunohost Git/Infra notifications> [yunohost] tituspijean created new branch enh-san
[21:47:43]
<Yunohost Git/Infra notifications> [vinaigrette] Salamandar pushed 1 commit to master: Fix shellcheck issues ([35bb21c4](https://github.com/YunoHost/vinaigrette/commit/35bb21c40f95b2eb3a8ed652f2f01b1e1e4c4586))
[21:49:02]
<Yunohost Git/Infra notifications> [yunohost] tituspijean opened [pull request #1994](https://github.com/YunoHost/yunohost/pull/1994): Support domains in cert_alternate_names hook
[21:49:51]
<Yunohost Git/Infra notifications> [vinaigrette] Salamandar pushed 1 commit to trixie: Add support for trixie builds ([d0a17eb7](https://github.com/YunoHost/vinaigrette/commit/d0a17eb761b32813a502e29dfddc87a66ac95326))
[21:50:07]
<Yunohost Git/Infra notifications> [vinaigrette] Salamandar pushed 2 commits to master ([35bb21c40f95...4a50c0b339db](https://github.com/YunoHost/vinaigrette/compare/35bb21c40f95...4a50c0b339db))
[21:50:08]
<Yunohost Git/Infra notifications> [vinaigrette] Salamandar merged [pull request #9](https://github.com/YunoHost/vinaigrette/pull/9): Add support for trixie builds
[21:50:09]
<Yunohost Git/Infra notifications> [vinaigrette/master] Add support for trixie builds - Félix Piédallu
[21:50:10]
<Yunohost Git/Infra notifications> [vinaigrette/master] Merge pull request #9 from YunoHost/trixie - Salamandar
[21:57:32]
<Yunohost Git/Infra notifications> [vinaigrette] alexAubin deleted branch trixie
[22:10:03]
<Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1526864643](https://gitlab.com/YunoHost/yunohost/-/pipelines/1526864643) failed on branch enh-san
[22:27:13]
<Yunohost Git/Infra notifications> [dynette] zamentur pushed 1 commit to enh-include: [enh] Write key in namedformat directly ([57d41487](https://github.com/YunoHost/dynette/commit/57d41487811849abb1f2e6453f0f0210020dffc3))
[22:29:04]
<Yunohost Git/Infra notifications> [dynette] zamentur edited [pull request #18](https://github.com/YunoHost/dynette/pull/18): [enh] Avoid to generate tons of lines to grant permissions
[22:32:31]
<Yunohost Git/Infra notifications> [dynette] zamentur edited [pull request #18](https://github.com/YunoHost/dynette/pull/18): [enh] Avoid to generate tons of lines to grant permissions