[00:46:12]
<Yunohost Git/Infra notifications> Job [#30122](https://ci-apps.yunohost.org/ci/job/30122) for n8n failed miserably :(
[01:18:17]
<Yunohost Git/Infra notifications> [package_check] alexAubin pushed 1 commit to master: Fix bash syntax check not properly reporting issues ([2fb7af99](https://github.com/YunoHost/package_check/commit/2fb7af995c867e9f5daa1a001bca6a5092effc29))
[01:20:54]
<Yunohost Git/Infra notifications> Job [#30128](https://ci-apps.yunohost.org/ci/job/30128) for paperless-ngx failed miserably :(
[01:21:00]
<Yunohost Git/Infra notifications> [package_check] alexAubin pushed 1 commit to master: curl tests: zzz app such as hextris includes http:// (no s..) to google fonts ... ([2b2fd1f9](https://github.com/YunoHost/package_check/commit/2b2fd1f9f4459c767b738820bf6acca747d4698c))
[01:21:12]
<Yunohost Git/Infra notifications> [package_check] alexAubin pushed 1 commit to master: curl tests: Again tweaking the debugging tails upon error code ? ([5b64c232](https://github.com/YunoHost/package_check/commit/5b64c232c5b3bac4948c1541bbb49c7f0c4e5f12))
[01:27:29]
<Yunohost Git/Infra notifications> [package_check] alexAubin pushed 1 commit to master: curl tests: aaaaand some assets url start with // ([32611255](https://github.com/YunoHost/package_check/commit/32611255a4a213b5b024a88a54976d99d6cc91ce))
[01:44:46]
<Yunohost Git/Infra notifications> [package_check] alexAubin pushed 1 commit to master: curl test: missing f for f-string ([553d7c6a](https://github.com/YunoHost/package_check/commit/553d7c6a763ffe55e04f02c9be070997a3db4761))
[01:46:31]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 28 pending update PRs
- 8 new apps PRs
- 5 failed apps updates: anarchism, epicyon, mlmmj, mobilizon, pleroma
See the full log here: http://paste.yunohost.org/raw/efequgopud
[02:29:33]
<Yunohost Git/Infra notifications> [package_check] alexAubin pushed 1 commit to master: curl tests: tail hanging forever to stdin ~_~ ([cf51d372](https://github.com/YunoHost/package_check/commit/cf51d37285a67a37c854ee56779c67b420c24d18))
[02:32:33]
<Aleks (he/him/il/lui)> > <@yunohostinfra:matrix.org> Autoupdater just ran, here are the results:
>
> - 28 pending update PRs
> - 8 new apps PRs
> - 5 failed apps updates: anarchism, epicyon, mlmmj, mobilizon, pleroma
>
> See the full log here: http://paste.yunohost.org/raw/efequgopud
not sure what's happening exactly but many of those follow the pattern `404 for https://<forge url>/api/v4/projects/$app%2F$app%2F`, sounds like some API change in gitlab or something
[03:38:07]
<Yunohost Git/Infra notifications> [apps] yunohost-bot created new branch add-to-wishlist-redlib
[03:38:08]
<Yunohost Git/Infra notifications> [apps] yunohost-bot pushed 1 commit to add-to-wishlist-redlib: Add Redlib to wishlist ([2cfe395a](https://github.com/YunoHost/apps/commit/2cfe395a8cc77b5fa1d5f6e7ea101702e82e834b))
[03:38:11]
<Yunohost Git/Infra notifications> [apps] yunohost-bot labeled Wishlist on [pull request #2552](https://github.com/YunoHost/apps/pull/2552): Add Redlib to wishlist
[03:38:11]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2552](https://github.com/YunoHost/apps/pull/2552): Add Redlib to wishlist
[04:34:37]
<Yunohost Git/Infra notifications> [apps] OniriCorpe [commented](https://github.com/YunoHost/apps/pull/2552#issuecomment-2323163084) on [issue #2552](https://github.com/YunoHost/apps/pull/2552) Add Redlib to wishlist: hm https://github.com/redlib-org/redlib?tab=readme-ov-file#how-is-it-different-from-other-reddit-front-ends
[08:51:59]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u pushed 1 commit to testing: Fix notify_push after reboot/restart the service, create the folder /var/run/nextcloud via systemd ([19ae3b02](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/19ae3b021f4395d4543181836aee1a1ce4932919))
[08:52:03]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/721#issuecomment-2323238806) on [issue #721](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/721) Testing: testme
[08:52:07]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/721#issuecomment-2323238819) on [issue #721](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/721) Testing: Fingers crossed
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/18889/ba...
[08:52:13]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/721#issuecomment-2323238820) on [issue #721](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/721) Testing: :worm:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bookworm-dev.yunohost.org/ci/api/job/2572/badg...
[09:25:54]
<orhtej2> > <@Alekswag:matrix.org> not sure what's happening exactly but many of those follow the pattern `404 for https://<forge url>/api/v4/projects/$app%2F$app%2F`, sounds like some API change in gitlab or something
There's an extra trailing slash url encoded for some reason
[10:53:45]
<Yunohost Git/Infra notifications> [apps] ericgaspar [commented](https://github.com/YunoHost/apps/pull/2552#issuecomment-2323284558) on [issue #2552](https://github.com/YunoHost/apps/pull/2552) Add Redlib to wishlist: Im using the Redlib source as a replacement for Libreddit -> https://github.com/YunoHost-Apps/libreddit_ynh/blob/923c2f...
[14:24:13]
<Aleks (he/him/il/lui)> dafuq, ci-apps-dev already "completed" the helpers-2.1 jobs o.O
[14:24:47]
<selfhoster1312> it deserves a cookie? 🤷♀️️
[15:05:29]
<Yunohost Git/Infra notifications> [package_check] alexAubin pushed 1 commit to master: Tweak basic bash syntax check to return level 0 such that yunorunner flags the job as failure instead of technical er... ([a98ec7d4](https://github.com/YunoHost/package_check/commit/a98ec7d4702bfabfab2e6752be3e8bd31637e4f9))
[15:06:22]
<Aleks (he/him/il/lui)> restarting yunorunner on ci-apps-dev because for some reason 2 workers disappeared ~_~
[15:13:35]
<Yunohost Git/Infra notifications> [package_check] alexAubin pushed 1 commit to master: shfmt ([45485f49](https://github.com/YunoHost/package_check/commit/45485f499ad2eeee125a0661edc38381f045fdf3))
[15:30:40]
<Yunohost Git/Infra notifications> [package_check] selfhoster1312 opened [pull request #171](https://github.com/YunoHost/package_check/pull/171): Recommend btrfs storage driver for better perf (default on Incus)
[15:40:08]
<Yunohost Git/Infra notifications> [package_check] alexAubin merged [pull request #171](https://github.com/YunoHost/package_check/pull/171): Recommend btrfs storage driver for better perf (default on Incus)
[15:40:10]
<Yunohost Git/Infra notifications> [package_check] alexAubin pushed 2 commits to master ([45485f499ad2...f3518f7dc6bb](https://github.com/YunoHost/package_check/compare/45485f499ad2...f3518f7dc6bb))
[15:55:50]
<Yunohost Git/Infra notifications> [package_check] alexAubin pushed 1 commit to master: bash syntax check: gotta return 0 for the job to be flagged as failure ([22d703cb](https://github.com/YunoHost/package_check/commit/22d703cb4805d1e7037f6189c11624d00bc2c2de))
[15:56:11]
<Yunohost Git/Infra notifications> [package_check] alexAubin pushed 1 commit to master: Hm i meant exit? ([79810671](https://github.com/YunoHost/package_check/commit/798106719326ca417565dcaebb4e26f69b13e775))
[16:06:23]
<Yunohost Git/Infra notifications> [package_check] alexAubin pushed 1 commit to master: Uhoh ([eff66e2a](https://github.com/YunoHost/package_check/commit/eff66e2aacf66bf99a5f49dad601128490387a67))
[16:09:59]
<Aleks (he/him/il/lui)> (╯°□°）╯︵ ┻━┻ hmgnngnn
[16:12:13]
<Yunohost Git/Infra notifications> [package_check] alexAubin pushed 1 commit to master: JSON use double quote and not simple quote, JSus.christ ([0ccab26a](https://github.com/YunoHost/package_check/commit/0ccab26a7dea9e18ff425f23d4ad864aa06111b7))
[16:13:02]
<Aleks (he/him/il/lui)> finally
[16:24:44]
<Yunohost Git/Infra notifications> Job [#30111](https://ci-apps.yunohost.org/ci/job/30111) for moodle failed miserably :(
[16:32:45]
<selfhoster1312> does the auto patch bot tries to apply a diff based on default branch to the testing branch? should it not ensure testing >= default then apply the patch on testing?
[16:33:42]
<Aleks (he/him/il/lui)> yes, that's because i didnt realize it would patch the master branch instead of testing, then I only realized this near the end of creating the 400ish PRs and battling with ratelimits
[16:34:23]
<Aleks (he/him/il/lui)> anyway it's totally ok to recreate a clean branch by applying the script manually : https://github.com/YunoHost/apps/tree/master/tools/helpers2.1
[16:34:46]
<selfhoster1312> meh, i just rebased and forced pushed on the bot's branch :D
[16:34:46]
<Aleks (he/him/il/lui)> (just don't open the code if you're afraid to lose sanity points)
[16:34:56]
<selfhoster1312> (i was specifically about to do this)
[16:54:47]
<selfhoster1312> btw Aleks (he/him/il/lui) if you haven't seen it i recommended watching a FOSDEM talk about Debian Janitor and best practices when writing a bot like this: https://archive.fosdem.org/2023/schedule/event/automation_debian/
it's really interesting ^^
[17:09:08]
<Aleks (he/him/il/lui)> >https://janitor.debian.net/
Error: Server Error
[17:09:13]
<Aleks (he/him/il/lui)> 😬
[17:09:16]
<selfhoster1312> woops :)
[17:09:19]
<selfhoster1312> still a great talk :D
[17:09:43]
<selfhoster1312> btw Aleks (he/him/il/lui) what's the spirit of ynh_exec_warn_less... now that it's been removed in new helpers shouldn't we just remove it in all cases in the autopatch?
[17:10:03]
<selfhoster1312> was it because at some point STDERR output meant actual warnings sent to the user?
[17:10:19]
<selfhoster1312> (is that still the case???)
[17:10:34]
<Aleks (he/him/il/lui)> it's called ynh_hide_warnings now and it's just a wrapper for 2>&1 (stderr redirect to stdout = debug flow)
[17:11:05]
<Aleks (he/him/il/lui)> the use case is running stuff like "npm install" or other stuff which dump a stupid amount of stuff to stderr, which is displayed as warnings to the user
[17:11:21]
<selfhoster1312> ok so in install scripts STDERR is considered a warning, is that what you're saying?
[17:11:34]
<Aleks (he/him/il/lui)> stuff like boring issues nobody cares about about dependencies-of-dependencies-of-dependencies-of-dependencies
[17:11:37]
<Aleks (he/him/il/lui)> yeah
[17:11:39]
<selfhoster1312> ok
[17:12:12]
<selfhoster1312> so beyond the `Unecessary exec_warn_less` the autopatch could just replace with ynh_hide_warnings
[17:12:14]
<Aleks (he/him/il/lui)> but then the thing is that, because many people simply copypasta code between apps, there's many occurences of `ynh_hide_warnings` which are uncessary
[17:12:29]
<selfhoster1312> yeah right
[17:12:30]
<Aleks (he/him/il/lui)> some of them got removed because there were now unecessary ... i don't remember what exactly
[17:12:43]
<Aleks (he/him/il/lui)> > <selfhoster1312> so beyond the `Unecessary exec_warn_less` the autopatch could just replace with ynh_hide_warnings
yes that's what it does ?
[17:13:03]
<selfhoster1312> ha yes
[17:14:27]
<selfhoster1312> and so about that yarn thing should i just trigger than CI again? or is there some point when it's updated that we haven't reached yet?
[17:15:50]
<Aleks (he/him/il/lui)> should be fixed, the CI was updated to 11.2.30 sometime yesterday
[17:15:54]
<selfhoster1312> ok thx
[17:15:58]
<Aleks (he/him/il/lui)> and anyway it should just be a warning iirc
[17:16:27]
<selfhoster1312> oh you're right that's not what triggered the actual error
[17:16:28]
<selfhoster1312> my bad
[17:31:11]
<selfhoster1312> so if i'm reading correctly the autopatch should check that ruby_version / node_version is defined in _common.sh and remove the arg to the helper?
[17:31:17]
<selfhoster1312> (i mean it's not doing that but should it?)
[17:34:06]
<Aleks (he/him/il/lui)> i don't think it magically add the ruby_version / node_version to _common.sh, i don't quite remember, this feels like i did this 6 months ago
[17:34:28]
<selfhoster1312> it's ok anyway if it's something so simple :)
[17:35:15]
<selfhoster1312> hmm i think autopatch is unhappy:
```
Some lines could not be parsed in script install. (That's probably not really critical)
? No closing quotation : ynh_exec_as_app ruby_load_path" $ld_preload bin/bundle config deployment 'true'
? No closing quotation : ynh_exec_as_app ruby_load_path" $ld_preload bin/bundle config without 'development test'
? No closing quotation : ynh_exec_as_app ruby_load_path" $ld_preload bin/bundle config set force_ruby_platform true
? No closing quotation : ynh_exec_as_app ruby_load_path" $ld_preload bin/bundle install -j$(getconf _NPROCESSORS_ONLN)
```
[17:43:21]
<selfhoster1312> (that's the CI after autopatch)
[17:50:49]
<Aleks (he/him/il/lui)> Yes, expected syntax issues
[17:51:15]
<selfhoster1312> less expected:
> 287237 INFO WARNING - ./install: line 85: ynh_add_swap: command not found
> https://ci-apps-bookworm-dev.yunohost.org/ci/job/2596
[18:01:29]
<Aleks (he/him/il/lui)> yeah whatever the bookworm ci image ain't up to date ..
[18:02:03]
<selfhoster1312> shit happens
[18:02:10]
<selfhoster1312> mean i can take my day off and do something else
[20:41:20]
<Yunohost Git/Infra notifications> [my_webapp_ynh] ShaAlexander [comment](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/d291edbc72f80e457a5966a5df376404c704cc07#commitcomment-146074231) on commit d291edb: why isnt this fixed in an update yet?
[20:56:52]
<Yunohost Git/Infra notifications> [my_webapp_ynh] grenagit opened [pull request #143](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/143): [Doc] Add PHP 8.3 in available versions