[01:50:56]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 30 pending update PRs
- 8 new apps PRs
See the full log here: http://paste.yunohost.org/raw/qojakabaxe
[03:01:18]
<Yunohost Git/Infra notifications> App pairdrop rises from level 7 to 8 in job [#29930](https://ci-apps.yunohost.org/ci/job/29930) !
[07:46:22]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to update_app_levels: Update apps.toml: revert jellysee to level 8 ([6fc9c3b7](https://github.com/YunoHost/apps/commit/6fc9c3b76da0054f5709cb08c05bd6bd9e97b50f))
[07:46:46]
<Yunohost Git/Infra notifications> [apps] alexAubin edited [pull request #2536](https://github.com/YunoHost/apps/pull/2536): Update app levels according to CI results
[11:46:12]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2536](https://github.com/YunoHost/apps/pull/2536): Update app levels according to CI results
[11:46:45]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([c7080e21](https://github.com/YunoHost/apps/commit/c7080e2101345ee68279905411fd753d997e8517))
[11:57:31]
<Yunohost Git/Infra notifications> App pihole failed all tests in job [#29938](https://ci-apps.yunohost.org/ci/job/29938) :(
[15:32:17]
<selfhoster1312> maybe obvious but from reading https://yunohost.org/en/packaging_config_panels it's not clear to me: how to declare some config panel questions part of install questions? because they're the same and can be changed after install
[15:40:08]
<Yunohost Git/Infra notifications> [apps] ericgaspar approved [pull request #2536](https://github.com/YunoHost/apps/pull/2536#pullrequestreview-2258769372) Update app levels according to CI results
[15:41:45]
<Aleks (he/him/il/lui)> that's part of "Clarify the whole story between config panels, app settings, and ynh_add_config" in https://github.com/YunoHost/issues/issues/2136
[15:54:11]
<selfhoster1312> ha thanks
[16:43:30]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch update_app_levels
[16:44:36]
<Yunohost Git/Infra notifications> [apps] ericgaspar changes_requested [pull request #2504](https://github.com/YunoHost/apps/pull/2504#pullrequestreview-2258785801) Adding Hotglue app
[16:44:37]
<Yunohost Git/Infra notifications> [apps] ericgaspar [commented](https://github.com/YunoHost/apps/pull/2504#discussion_r1730040470) on pull request #2504 Adding Hotglue app: suggestion url = "https://github.com/YunoHost-Apps/hotglue_ynh"
[16:45:08]
<Yunohost Git/Infra notifications> [apps] ericgaspar approved [pull request #2504](https://github.com/YunoHost/apps/pull/2504#pullrequestreview-2258785914) Adding Hotglue app
[16:45:15]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to master: Adding Hotglue app (#2504) * Adding Hotglue app * Adding Hotglue logo * Update apps.toml --------- Co-authored-by: ... ([1a528d55](https://github.com/YunoHost/apps/commit/1a528d55a5adc5517ed5b6448ad1fd7516917382))
[16:45:20]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #2504](https://github.com/YunoHost/apps/pull/2504): Adding Hotglue app
[17:00:20]
<selfhoster1312> does README bot now rewrite commit history?
[17:00:39]
<selfhoster1312> i see my PRs now have a "Auto-update README" commit https://github.com/YunoHost-Apps/reverseproxy_ynh/pull/10
[17:00:59]
<selfhoster1312> i was confused so i rebased on testing to be sure, it was not there when i submitted the PR, and now here it is
[17:01:24]
<Aleks (he/him/il/lui)> it pushes the commit to the branch, yes
[17:01:32]
<selfhoster1312> ok :)
[17:03:47]
<Aleks (he/him/il/lui)> and then `git pull --rebase` ftw
[17:06:15]
<selfhoster1312> i love rebasing
[17:06:30]
<selfhoster1312> who needs merge commits polluting your history when you can have granular commits telling the whole story? :)
[17:07:08]
<selfhoster1312> https://xmpp-upload.kl.netlib.re/upload/VyJjYph9RQ2J9228/f9c4e73c65afd654562aabb00017b55cfbde586c0e6d20fd07382d2eaa88bdf7_1.jpg
[17:38:09]
<Yunohost Git/Infra notifications> [package_check] alexAubin pushed 1 commit to curl-tests: Rework test that app can be accessed ... add a new syntax in tests.toml to declare what URL to test, with/without being ... ([988ec3cf](https://github.com/YunoHost/package_check/commit/988ec3cf391af47cc8dff0aea1835f2eed490c04))
[17:38:10]
<Yunohost Git/Infra notifications> [package_check] alexAubin created new branch curl-tests
[17:39:00]
<Aleks (he/him/il/lui)> sooo i have something somewhat working that needs to be tested on the battlefield (regarding testing that several app's URLs return the expected content)
[17:39:20]
<Aleks (he/him/il/lui)> will deploy this on ci-apps-bookworm-dev
[17:43:32]
<Yunohost Git/Infra notifications> [nextcloud_ynh] alexAubin created new branch curl-tests
[17:43:37]
<Yunohost Git/Infra notifications> [nextcloud_ynh] alexAubin pushed 1 commit to curl-tests: Add experimental curl tests ([18567290](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/1856729055853c88fd9ed3580de0eabc00e6b32e))
[17:43:53]
<Yunohost Git/Infra notifications> [nextcloud_ynh] alexAubin opened [pull request #719](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/719): WIP: Experimental curl tests
[17:44:24]
<Yunohost Git/Infra notifications> [nextcloud_ynh] alexAubin edited [pull request #719](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/719): WIP: Experimental curl tests
[17:44:28]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/719#issuecomment-2308468743) on [issue #719](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/719) WIP: Experimental curl tests: :bug:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bookworm-dev.yunohost.org/ci/api/job/2241/badge...
[17:47:17]
<selfhoster1312> hmm you mean actual tests beyond "app homepage return 200"?
[17:47:23]
<selfhoster1312> that would be great!
[17:47:36]
<Aleks (he/him/il/lui)> yup
[17:47:47]
<Aleks (he/him/il/lui)> cf https://github.com/YunoHost-Apps/nextcloud_ynh/commit/1856729055853c88fd9ed3580de0eabc00e6b32e
[17:50:09]
<selfhoster1312> paradoxically i would argue integration tests is precisely where a complete script would be great and declarative can't go far enough :) :)
[17:50:35]
<selfhoster1312> but that looks good! maybe expect_status or expect_header would be nice?
[17:50:42]
<selfhoster1312> good luck, and may the CI gods be with you <3
[17:52:10]
<selfhoster1312> can i try it on reverseproxy_ynh too?
[17:53:00]
<selfhoster1312> (and in that case would it be possible to start a simple script such as python -m http.server before running the tests to make sure the reverse proxying works?)
[17:54:48]
<Aleks (he/him/il/lui)> i guess yeah, you could add this to the preinstall stuff
[17:55:20]
<Aleks (he/him/il/lui)> cf `preinstall = ` in https://github.com/YunoHost/package_check
[17:56:35]
<Yunohost Git/Infra notifications> [nextcloud_ynh] alexAubin pushed 1 commit to curl-tests: curl tests: fix expect_title lang ([efeea41d](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/efeea41ddffe868578ba4323d645d823fcbea318))
[17:56:42]
<selfhoster1312> so preinstall is only setup for tests?
[17:58:10]
<Aleks (he/him/il/lui)> yup
[18:07:34]
<Yunohost Git/Infra notifications> [package_check] alexAubin pushed 1 commit to curl-tests: Fix noslash test ([0739f61f](https://github.com/YunoHost/package_check/commit/0739f61fedb67e20b88e5a857037a2166647a424))
[21:31:29]
<selfhoster1312> ```
./lib/common.sh: line 75: warn: command not found
```
[21:31:33]
<selfhoster1312> in package_check
[21:32:57]
<selfhoster1312> i see warn but not where it's defined?