[16:24:25]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to master: metrics: move the metric start/stop around the install/upgrade/restore command, because we want to avoid being polluted ... ([f5148242](https://github.com/YunoHost/package_check/commit/f514824235bb65a86edc2b24014898368b77e2f8))
[16:24:41]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to master: metrics: fix echo flooding the logs + improve messaging + report diff between start and end, not just peak ([f35f74ae](https://github.com/YunoHost/package_check/commit/f35f74ae2ec7b1e9b654ed2b11752d8aeb3909c7))
[16:24:43]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/535#issuecomment-1438699591) on [issue #535](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/535) opcache.interned_strings_buffer higher 8: > a server with 32GB of RAM i guess Yes sorry to not be accurate: a 32GB ram server Here an answer from [MichaIng](htt...
[16:24:43]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to master: Lower the warning count complaining threshold now that postgresql warnings are hidden ([fe56eb2d](https://github.com/YunoHost/package_check/commit/fe56eb2de7c0d3fdcc43ab779ab79746875e7838))
[17:11:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @kicroc2000 [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/554#issuecomment-1438827114) on [issue #554](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/554) upgrade 22.2.10 ->25.0.3 error : PHP version: Hello Eric, fulltextsearch deactivated, and all went smoothly. Thank you very much for your help regards
[17:12:30]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar closed [issue #554](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/554): upgrade 22.2.10 ->25.0.3 error : PHP version
[21:10:12]
<tituspijean> In https://ci-apps-dev.yunohost.org/ci/job/5762:
```
Traceback (most recent call last):
File "<string>", line 1, in <module>
AssertionError: Missing default value
Running: yunohost app install --force /app_folder -a "domain=sub.domain.tld&path=/&admin=package_checker&password=Passw0rd!&title=My forum&init_main_permission=visitors&language=fr&swap="
```
is it complaining of `swap` not having a default value, even though it is an "alert" type?
[21:16:08]
<Aleks (he/him/il/lui)> eeeeh
[21:16:09]
<Aleks (he/him/il/lui)> didnt really think about that case i guess haha
[21:16:18]
<Aleks (he/him/il/lui)> and anyway it should be acceptable to have no default values
[21:17:03]
<tituspijean> Anyhow, I can move that alert in pre_install.md 😉
[21:18:18]
<Aleks (he/him/il/lui)> hmmmyeah i think that's the right thing to do
[21:24:49]
<tituspijean> are the markdown filenames case-sensitive?
[21:26:10]
<tituspijean> looks like the linter advice about pre_install.md and co should be capitalized
[21:27:54]
<tituspijean> mmh, the RAM monitoring is not shown in the test
[21:30:46]
<tituspijean> actually no, the linter mentions they should be in `doc/notifications`, the doc says `doc/` only
[21:40:37]
<tituspijean> `__APP__` variable does not seem to be replaced 😕
[21:40:38]
<tituspijean> I only seem to be complaining here, so let me compensate: oh gosh packaging v2 is so great 🤟
[22:06:04]
<Yunohost Git/Infra notifications> [example_ynh] @zamentur [commented](https://github.com/YunoHost/example_ynh/pull/201#discussion_r1113616529) on pull request #201 Simplify change_url syntax: i think nginx reload should be done after service restart... to avoid 502
[22:08:42]
<Yunohost Git/Infra notifications> [package_check] @OniriCorpe edited [issue #140](https://github.com/YunoHost/package_check/issues/140): [enh] Test config panels
[22:10:10]
<Yunohost Git/Infra notifications> [package_linter] @tituspijean pushed 1 commit to fix-v2: Fix location of PRE/POST_INSTALL.md files ([880cfd86](https://github.com/YunoHost/package_linter/commit/880cfd86682693f80d7e2402768a26948e6d7f53))
[22:10:10]
<Yunohost Git/Infra notifications> [package_linter] @tituspijean created new branch fix-v2
[22:10:12]
<Yunohost Git/Infra notifications> [package_linter] @tituspijean opened [pull request #110](https://github.com/YunoHost/package_linter/pull/110): Fix location of PRE/POST_INSTALL.md files
[22:10:14]
<tituspijean> > <@titus:pijean.ovh> actually no, the linter mentions they should be in `doc/notifications`, the doc says `doc/` only
^
[22:51:24]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 2 commits to v2 ([3b4108eac2c1...80000fc87db1](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/3b4108eac2c1...80000fc87db1))
[22:51:25]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2] Update _common.sh - Éric Gaspar
[22:51:25]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2: Update upgrade ([3b4108ea](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/3b4108eac2c16552ceb313d66fcec7601e007d76))
[22:51:25]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2: fix ([3fa4fadf](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/3fa4fadf419ddf0523f95b0353bc955628b76db6))
[22:51:25]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2] Update tests.toml - Éric Gaspar
[22:51:40]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2: Update install ([71a4b662](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/71a4b662dc270e1dfa266ed1be88bd62098aadd8))
[23:12:36]
<tituspijean> Playing with PyGithub to report CI statuses: https://github.com/YunoHost-Apps/flarum_ynh/pull/208
[23:21:13]
<Aleks (he/him/il/lui)> oooooooooh
[23:21:15]
<Aleks (he/him/il/lui)> hell yea
[23:21:59]
<tituspijean> erf, it looks like one cannot report a warning as a status (like "it's a success, but a lower level than previously")
[23:22:32]
<Aleks (he/him/il/lui)> ah it's either success or failure ?
[23:22:53]
<tituspijean> yup
> Can be one of: error, failure, pending, success
[23:23:26]
<Aleks (he/him/il/lui)> :|
[23:26:51]
<tituspijean> ah I'm using the Checks API, maybe a Check Run is better suited