[00:01:26]
<denisapain> (using the `&` character)
[00:14:44]
<orhtej2> > <@denisapain:matrix.org> only thing left is the systemd service since I need to start both backend and frontend, and the current yarn command only starts the backend, also I translated the readme to french with deepl, I hope its decent
just add 2 systemd units
[00:18:59]
<denisapain> > just add 2 systemd units
okay, but might be tomorrow since its 2am
[01:13:48]
<denisapain> how though?
the code that adds the one systemd is
`ynh_config_add_systemd`
I'm not really sure how to make it add another file
[01:14:09]
<denisapain> oh wait
[01:14:12]
<denisapain> i think i might got it
[01:15:47]
<denisapain> > just add 2 systemd units
how though?
[01:16:10]
<denisapain> `ynh_config_add_systemd` is how it adds systemd.service, I don't really know how I would make it add multiple
[01:17:25]
<Émy – OniriCorpe> By providing the two arguments
[01:17:32]
<Émy – OniriCorpe> `ynh_add_systemd_config [--service=service] [--template=template]`
[01:18:03]
<Émy – OniriCorpe> https://yunohost.org/en/packaging_apps_helpers#ynh-add-systemd-config
[01:20:36]
<Émy – OniriCorpe> So it could be:
`ynh_add_systemd_config --service= "$app-backend" --template="$app-backend"`
and
`ynh_add_systemd_config --service= "$app-frontend" --template="$app-frontend"`
[01:21:11]
<Émy – OniriCorpe> Or one with just `ynh_add_systemd_config` and the other with the arguments
[08:10:23]
<Yunohost Git/Infra notifications> yunohost-bot forked apps_tools to [yunohost-bot/apps_tools](https://github.com/yunohost-bot/apps_tools)
[08:16:32]
<Yunohost Git/Infra notifications> [apps_tools] yunohost-bot opened [pull request #15](https://github.com/YunoHost/apps_tools/pull/15): Translations update from Weblate
[09:44:32]
<eric_G> web authentication (for owncast, etherpad...) seems not to be working with YNH 12 as it was with YNH 11. what changed, how can we fix that?
[10:49:10]
<kayou> set `protect_against_basic_auth_spoofing` to false in manifest
[10:49:38]
<kayou> like [this](https://github.com/YunoHost-Apps/gitlab_ynh/blob/c3ddaab5cd6d46bbce68d04d04db5c6ddd861436/manifest.toml#L77)
[11:04:49]
<eric_G> mmh... I still can't connect with this setting
[11:05:09]
<eric_G> `401 Unauthorized`
[12:26:48]
<kayou> idk, maybe with `auth_header = False` too? :/
[12:30:12]
<Yunohost Git/Infra notifications> [package_linter] Krakinou opened [issue #166](https://github.com/YunoHost/package_linter/issues/166): False positive? utf-8 codec cant decode byte 0x80 in position 99: invalid start byte
[12:47:22]
<Yunohost Git/Infra notifications> [package_linter] Salamandar [commented](https://github.com/YunoHost/package_linter/issues/166#issuecomment-2480549041) on [issue #166](https://github.com/YunoHost/package_linter/issues/166) False positive? utf-8 codec cant decode byte 0x80 in position 99: invalid start byte : Yeah, the linter expects "config" files to be pure text, not binary… so of course when trying to decode with utf-8, it b...
[12:54:05]
<Yunohost Git/Infra notifications> [package_linter] Krakinou [commented](https://github.com/YunoHost/package_linter/issues/166#issuecomment-2480550741) on [issue #166](https://github.com/YunoHost/package_linter/issues/166) False positive? utf-8 codec cant decode byte 0x80 in position 99: invalid start byte : make sense. Should move in /sources?
[13:05:13]
<Yunohost Git/Infra notifications> [package_linter] Salamandar created new branch fix_imports
[13:05:14]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 1 commit to fix_imports: Remove glob imports ([4fa7ceb2](https://github.com/YunoHost/package_linter/commit/4fa7ceb2c3f29febf091cfeec27ecc66541604e5))
[13:05:48]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 1 commit to fix_imports: Remove glob imports ([c97cff00](https://github.com/YunoHost/package_linter/commit/c97cff000e6a6153f9a9731c0f9ad3520898637f))
[13:19:57]
<Yunohost Git/Infra notifications> [package_linter] Salamandar opened [pull request #167](https://github.com/YunoHost/package_linter/pull/167): Remove glob imports for better readability and linting
[13:20:26]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 1 commit to master: Remove glob imports ([f9791b13](https://github.com/YunoHost/package_linter/commit/f9791b13380ddad2aac91c7eb90b824287bdcd3d))
[13:20:26]
<Yunohost Git/Infra notifications> [package_linter] Salamandar merged [pull request #167](https://github.com/YunoHost/package_linter/pull/167): Remove glob imports for better readability and linting
[13:20:27]
<Yunohost Git/Infra notifications> [package_linter] Salamandar deleted branch fix_imports
[13:23:01]
<Yunohost Git/Infra notifications> [package_linter] Salamandar created new branch fix_nontext_conf
[13:23:05]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 1 commit to fix_nontext_conf: When checking conf files, we need to ignore files that do not look like text files. In the whole catalog, we never saw ... ([4c69c72a](https://github.com/YunoHost/package_linter/commit/4c69c72a3df3d0701448c6c7df46a8fa47fad26c))
[13:23:05]
<Yunohost Git/Infra notifications> [package_linter] Salamandar opened [pull request #168](https://github.com/YunoHost/package_linter/pull/168): When checking conf files, we need to ignore files that do not look like text files
[13:23:26]
<Yunohost Git/Infra notifications> [package_linter] Salamandar edited [pull request #168](https://github.com/YunoHost/package_linter/pull/168): When checking conf files, we need to ignore files that do not look like text files
[13:25:39]
<Aleks (he/him/il/lui)> > <@yunohostinfra:matrix.org> [package_linter] Salamandar opened [pull request #168](https://github.com/YunoHost/package_linter/pull/168): When checking conf files, we need to ignore files that do not look like text files
I did ... the point is that idk at some point some people were editing file with Windows I'm guessing which changed the encoding somehow and then it created random explosions and whatever
[13:26:02]
<Aleks (he/him/il/lui)> (encoded in iso-9981whatever instead of utf-8)
[13:27:00]
<eric_G> > <@kayou:matrix.org> idk, maybe with `auth_header = False` too? :/
did that too...
[13:27:20]
<Yunohost Git/Infra notifications> [apps] mdtita opened [pull request #2706](https://github.com/YunoHost/apps/pull/2706): headplane_ynh
[13:27:33]
<Aleks (he/him/il/lui)> > <@kayou:matrix.org> like [this](https://github.com/YunoHost-Apps/gitlab_ynh/blob/c3ddaab5cd6d46bbce68d04d04db5c6ddd861436/manifest.toml#L77)
uuuh what
[13:27:40]
<Aleks (he/him/il/lui)> that doesn't work ?
[13:27:52]
<Aleks (he/him/il/lui)> `protect_against_basic_auth_spoofing` is supposed to be an app setting @_@
[13:29:52]
<eric_G> > <@Alekswag:matrix.org> that doesn't work ?
not with owncast
[13:30:01]
<eric_G> nor etherpad
[13:30:33]
<kayou> > <@Alekswag:matrix.org> `protect_against_basic_auth_spoofing` is supposed to be an app setting @_@
what, it's not a property of a permission? O.O
[13:30:37]
<Aleks (he/him/il/lui)> (i mean it's expected that it wont work because the property doesnt do anything, it needs to be an app setting)
[13:31:00]
<Aleks (he/him/il/lui)> cf https://github.com/YunoHost/yunohost/blob/dev/src/app.py#L1725
[13:31:52]
<kayou> crap
[13:32:18]
<Aleks (he/him/il/lui)> and https://github.com/YunoHost/issues/issues/2422
[13:32:38]
<Aleks (he/him/il/lui)> which requires https://github.com/YunoHost/yunohost/pull/1917
[13:33:44]
<kayou> `auth_header` is also an app setting @.@
[13:34:18]
<Aleks (he/him/il/lui)> yes and if we want to add another flag we need to have a stupid fucking migration for the LDAP database schema which is just ridiculous
[13:34:36]
<kayou> yeah i saw in your comment
[13:37:40]
<kayou> indeed, i don't have `protect_against_basic_auth_spoofing` in my ssowat conf, but I do have `auth_header=false` in my gitlab permission
[13:37:44]
<kayou> O.O
[13:39:19]
<eric_G> Ok, so what I understood was that it’s getting more complicated than `*** = False` 😅
[13:39:20]
<kayou> https://github.com/YunoHost/yunohost/blob/5c40410e3152126b54a48ea95fe9bedaa4d13b82/src/app.py#L1692
[13:39:52]
<kayou> yes, because `perm_info["auth_header"]` if false here
[13:41:32]
<kayou> > <@ericg:matrix.org> Ok, so what I understood was that it’s getting more complicated than `*** = False` 😅
i'd say if it doesn't work with `auth_header = false` like [this](https://github.com/YunoHost-Apps/peertube_ynh/blob/4dca5d5b09387f25799208afe35927bb6d00d484/manifest.toml#L66), you can try something like [this](https://github.com/YunoHost-Apps/gitea_ynh/blob/392d23c4ae6d5bf05ccec357d7e79bcc756c24bb/scripts/upgrade#L6) `ynh_app_setting_set --key=protect_against_basic_auth_spoofing --value=false`
[13:42:15]
<kayou> > <@kayou:matrix.org> yes, because `perm_info["auth_header"]` if false here
it's a bug, but it works as expected, kek
[13:43:10]
<Salamandar> > <@Alekswag:matrix.org> I did ... the point is that idk at some point some people were editing file with Windows I'm guessing which changed the encoding somehow and then it created random explosions and whatever
yeah but do we want to support this nightmare ?
[13:43:31]
<Aleks (he/him/il/lui)> to support ... not having to do support ? 😬
[13:44:01]
<Aleks (he/him/il/lui)> pretty sure sooner or later if we remove that check somebody is going to edit some file from windows and we're back to stupid random encoding issues
[13:44:54]
<Aleks (he/him/il/lui)> or maybe i'm mistaking with another check
[13:44:56]
<eric_G> > <@kayou:matrix.org> i'd say if it doesn't work with `auth_header = false` like [this](https://github.com/YunoHost-Apps/peertube_ynh/blob/4dca5d5b09387f25799208afe35927bb6d00d484/manifest.toml#L66), you can try something like [this](https://github.com/YunoHost-Apps/gitea_ynh/blob/392d23c4ae6d5bf05ccec357d7e79bcc756c24bb/scripts/upgrade#L6) `ynh_app_setting_set --key=protect_against_basic_auth_spoofing --value=false`
Thanks! that worked
[13:45:19]
<Salamandar> > <@Alekswag:matrix.org> pretty sure sooner or later if we remove that check somebody is going to edit some file from windows and we're back to stupid random encoding issues
aaaah i see what you mean, but
[13:45:28]
<Salamandar> IMHO we should only want utf-8 files
[13:46:02]
<Aleks (he/him/il/lui)> ah i'm confused with https://github.com/YunoHost/package_linter/blob/f9791b13380ddad2aac91c7eb90b824287bdcd3d/tests/test_app.py#L619
[13:46:07]
<Salamandar> so yeah we should do `read(encoding="utf-8")` and if there are encoding errors, assume it's a binary file (we have issues with PNGs, seed DB files etc)
[13:46:24]
<Salamandar> aaah i see
[13:46:44]
<Salamandar> did not know this test, and yeah 100% agree with you on this, unicode is the only extended encoding out there
[13:48:10]
<kayou> > <@ericg:matrix.org> Thanks! that worked
which one auth_header or protect?
[13:48:20]
<Aleks (he/him/il/lui)> (i'm trying to understand PR 168 😅)
[13:48:35]
<Salamandar> > <@Alekswag:matrix.org> (i'm trying to understand PR 168 😅)
it's because of https://github.com/YunoHost/package_linter/issues/166
[13:48:42]
<eric_G> > <@kayou:matrix.org> which one auth_header or protect?
protect
[13:48:43]
<Salamandar> when there are binary files in conf/, the linter fails
[13:48:50]
<Salamandar> > ! Can't open/read /tmp/package_check.EbTwIg/app_folder/conf/metadata.db.empty: 'utf-8' codec can't decode byte 0x80 in position 99: invalid start byte
[13:49:03]
<Aleks (he/him/il/lui)> ah, `conf/metadata.db.empty`
[13:49:07]
<Salamandar> yeah
[13:49:09]
<Aleks (he/him/il/lui)> hmmm
[13:49:15]
<eric_G> but I had also `auth_header` set to False
[13:49:24]
<Aleks (he/him/il/lui)> lgtm
[13:49:24]
<Aleks (he/him/il/lui)> and this should be an info and not a warning, yeah
[13:49:27]
<Salamandar> so imho if the read() returns a unicode error, we can just assume it's not text and not handle this file
[13:49:31]
<Salamandar> wait
[13:49:36]
<Salamandar> i forgot a return i think
[13:49:39]
<Salamandar> or `continue`
[13:49:49]
<Salamandar> yea
[13:49:55]
<Aleks (he/him/il/lui)> yield ? 👌
[13:50:01]
<Aleks (he/him/il/lui)> ah yes
[13:50:09]
<Salamandar> also the second `except` has return but it should be continue too
[13:50:18]
<Salamandar> because we don't test other conf files…
[13:50:19]
<Aleks (he/him/il/lui)> :O
[13:50:50]
<Aleks (he/him/il/lui)> sounds like a bug ?
[13:51:01]
<Aleks (he/him/il/lui)> ah yes read too quick
[13:51:22]
<Salamandar> that's why i don't like big `for` loop bodies
[13:51:27]
<Yunohost Git/Infra notifications> [package_linter] alexAubin approved [pull request #168](https://github.com/YunoHost/package_linter/pull/168#pullrequestreview-2440372291) When checking conf files, we need to ignore files that do not look like text files
[13:51:36]
<Salamandar> we easily forget we are in a for loop and just call return instead of continue
[13:52:45]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 1 commit to fix_nontext_conf: Fix except in for loops: missing continue, and replace return with continue ([cb755a56](https://github.com/YunoHost/package_linter/commit/cb755a568774e05dec740a66911714cbf370e195))
[13:52:48]
<Salamandar> pushed a second commit to fix that
[13:53:00]
<Aleks (he/him/il/lui)> (MFW people are talking about "official apps" https://i.imgflip.com/9akjq2.jpg )
[13:53:48]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 3 commits to master ([f9791b13380d...e32bee41d2df](https://github.com/YunoHost/package_linter/compare/f9791b13380d...e32bee41d2df))
[13:53:48]
<Yunohost Git/Infra notifications> [package_linter/master] When checking conf files, we need to ignore files that do not look like text files. In the whole catalog, we never saw ... - Félix Piédallu
[13:53:48]
<Yunohost Git/Infra notifications> [package_linter] Salamandar merged [pull request #168](https://github.com/YunoHost/package_linter/pull/168): When checking conf files, we need to ignore files that do not look like text files
[13:53:49]
<Yunohost Git/Infra notifications> [package_linter/master] Merge pull request #168 from YunoHost/fix_nontext_conf - Salamandar
[13:53:49]
<Yunohost Git/Infra notifications> [package_linter/master] Fix except in for loops: missing continue, and replace return with continue - Félix Piédallu
[13:53:49]
<Yunohost Git/Infra notifications> [package_linter] Salamandar closed [issue #166](https://github.com/YunoHost/package_linter/issues/166): False positive? utf-8 codec cant decode byte 0x80 in position 99: invalid start byte
[13:53:50]
<Yunohost Git/Infra notifications> [package_linter] Salamandar deleted branch fix_nontext_conf
[13:54:23]
<Yunohost Git/Infra notifications> [package_linter] Salamandar [commented](https://github.com/YunoHost/package_linter/issues/166#issuecomment-2480578226) on [issue #166](https://github.com/YunoHost/package_linter/issues/166) False positive? utf-8 codec cant decode byte 0x80 in position 99: invalid start byte : No, we just fixed our linter to ignore files that do not look like ascii/unicode files.
[13:54:40]
<Yunohost Git/Infra notifications> [package_linter] Salamandar [commented](https://github.com/YunoHost/package_linter/issues/166#issuecomment-2480578226) on [issue #166](https://github.com/YunoHost/package_linter/issues/166) False positive? utf-8 codec cant decode byte 0x80 in position 99: invalid start byte : No, we just fixed our linter to ignore files that do not look like ascii/unicode files. Other apps also have this issue ...
[13:57:45]
<Yunohost Git/Infra notifications> [package_linter] Salamandar [commented](https://github.com/YunoHost/package_linter/pull/111#issuecomment-2480579156) on [issue #111](https://github.com/YunoHost/package_linter/pull/111) Try to use tomllib and fallback to tomli: Thank you for this PR, and now that we are fully on Bookworm we can just use tomllib without falling back on tomli. Ill...
[14:00:03]
<Yunohost Git/Infra notifications> [apps] ericgaspar [commented](https://github.com/YunoHost/apps/pull/2706#issuecomment-2480579955) on [issue #2706](https://github.com/YunoHost/apps/pull/2706) headplane_ynh: you should clean up comments 🗑️
[14:01:28]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 1 commit to use_tomllib: Use the new standard tomllib library It was introduced in python3.11, available on bookworm ([dd58dbc3](https://github.com/YunoHost/package_linter/commit/dd58dbc321c10b0d515b2df69cade12604665958))
[14:01:28]
<Yunohost Git/Infra notifications> [package_linter] Salamandar created new branch use_tomllib
[14:01:37]
<Yunohost Git/Infra notifications> [package_linter] Salamandar opened [pull request #169](https://github.com/YunoHost/package_linter/pull/169): Use the new standard tomllib library
[14:01:46]
<Yunohost Git/Infra notifications> [package_linter] Salamandar closed [pull request #111](https://github.com/YunoHost/package_linter/pull/111): Try to use tomllib and fallback to tomli
[14:01:46]
<Yunohost Git/Infra notifications> [package_linter] Salamandar [commented](https://github.com/YunoHost/package_linter/pull/111#issuecomment-2480584510) on [issue #111](https://github.com/YunoHost/package_linter/pull/111) Try to use tomllib and fallback to tomli: See https://github.com/YunoHost/package_linter/pull/169
[14:03:40]
<Yunohost Git/Infra notifications> [apps] mdtita [commented](https://github.com/YunoHost/apps/pull/2706#issuecomment-2480585027) on [issue #2706](https://github.com/YunoHost/apps/pull/2706) headplane_ynh: > you should clean up comments 🗑️ done
[14:03:40]
<Yunohost Git/Infra notifications> [apps] mdtita closed [pull request #2706](https://github.com/YunoHost/apps/pull/2706): headplane_ynh
[14:03:43]
<Yunohost Git/Infra notifications> [apps] mdtita reopened [pull request #2706](https://github.com/YunoHost/apps/pull/2706): headplane_ynh
[14:06:48]
<Yunohost Git/Infra notifications> [apps] ericgaspar [commented](https://github.com/YunoHost/apps/pull/2706#issuecomment-2480585960) on [issue #2706](https://github.com/YunoHost/apps/pull/2706) headplane_ynh: add also, place Headplane in the alphabetical list (after headphones)
[14:10:54]
<Yunohost Git/Infra notifications> [apps] ericgaspar [commented](https://github.com/YunoHost/apps/pull/2706#issuecomment-2480587548) on [issue #2706](https://github.com/YunoHost/apps/pull/2706) headplane_ynh: You are highly encouraged to move headplane_ynh to the [YunoHost-app](https://github.com/yunohost-apps) repo
[14:15:03]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 1 commit to master: fix 4c69c72a: missing string formatting on message ([61fceb01](https://github.com/YunoHost/package_linter/commit/61fceb01e386315f358e9c24d95268793fc6ffc5))
[14:15:55]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 2 commits to use_tomllib ([dd58dbc321c1...34cb83ffd6dc](https://github.com/YunoHost/package_linter/compare/dd58dbc321c1...34cb83ffd6dc))
[14:15:56]
<Yunohost Git/Infra notifications> [package_linter/use_tomllib] Use the new standard tomllib library It was introduced in python3.11, available on bookworm - Jens Diemer
[14:15:56]
<Yunohost Git/Infra notifications> [package_linter/use_tomllib] tomllib requires binary open() - Félix Piédallu
[14:16:29]
<Yunohost Git/Infra notifications> [apps] mdtita [commented](https://github.com/YunoHost/apps/pull/2706#issuecomment-2480588999) on [issue #2706](https://github.com/YunoHost/apps/pull/2706) headplane_ynh: > You are highly encouraged to move headplane_ynh to the [YunoHost-app](https://github.com/yunohost-apps) repo sorry,...
[14:16:36]
<Yunohost Git/Infra notifications> [package_linter] Salamandar edited [pull request #169](https://github.com/YunoHost/package_linter/pull/169): Use the new standard tomllib library
[14:16:45]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 3 commits to master ([61fceb01e386...675a1fb83f94](https://github.com/YunoHost/package_linter/compare/61fceb01e386...675a1fb83f94))
[14:16:45]
<Yunohost Git/Infra notifications> [package_linter] Salamandar merged [pull request #169](https://github.com/YunoHost/package_linter/pull/169): Use the new standard tomllib library
[14:16:46]
<Yunohost Git/Infra notifications> [package_linter/master] Use the new standard tomllib library It was introduced in python3.11, available on bookworm - Jens Diemer
[14:16:46]
<Yunohost Git/Infra notifications> [package_linter/master] Merge pull request #169 from YunoHost/use_tomllib - Salamandar
[14:16:46]
<Yunohost Git/Infra notifications> [package_linter/master] tomllib requires binary open() - Félix Piédallu
[14:16:47]
<Yunohost Git/Infra notifications> [package_linter] Salamandar deleted branch use_tomllib
[14:51:09]
<eric_G> do we have a clue on this failing test? https://ci-apps-dev.yunohost.org/ci/job/4091
[14:51:36]
<Yunohost Git/Infra notifications> [apps] mdtita [commented](https://github.com/YunoHost/apps/pull/2706#issuecomment-2480588999) on [issue #2706](https://github.com/YunoHost/apps/pull/2706) headplane_ynh: > You are highly encouraged to move headplane_ynh to the [YunoHost-app](https://github.com/yunohost-apps) repo sorry,...
[14:52:06]
<Yunohost Git/Infra notifications> [package_linter] Salamandar created new branch use_tomllib
[14:52:06]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 2 commits to use_tomllib ([da5d7fa85599^...0b606f3eea37](https://github.com/YunoHost/package_linter/compare/da5d7fa85599^...0b606f3eea37))
[14:52:06]
<Yunohost Git/Infra notifications> [package_linter/use_tomllib] Add typing to lib - Félix Piédallu
[14:52:07]
<Yunohost Git/Infra notifications> [package_linter/use_tomllib] we changed the behaviour of urlopen() for easier typing - Félix Piédallu
[14:52:27]
<Yunohost Git/Infra notifications> [package_linter] Salamandar created new branch typing
[14:52:27]
<Yunohost Git/Infra notifications> [package_linter] Salamandar deleted branch use_tomllib
[14:56:30]
<Yunohost Git/Infra notifications> [apps] ericgaspar [commented](https://github.com/YunoHost/apps/pull/2706#issuecomment-2480604356) on [issue #2706](https://github.com/YunoHost/apps/pull/2706) headplane_ynh: you should be able to upload the app to the branch https://github.com/YunoHost-Apps/headplane_ynh
[15:02:01]
<Salamandar> > <@ericg:matrix.org> do we have a clue on this failing test? https://ci-apps-dev.yunohost.org/ci/job/4091
yeah ugh but actually no
[15:02:20]
<Salamandar> it's related to https://github.com/YunoHost/package_check/pull/178
[15:02:26]
<Salamandar> it fixed a huge bunch of apps but broke 3
[15:50:38]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 1 commit to typing: Fix most typing issues ([026ac87f](https://github.com/YunoHost/package_linter/commit/026ac87ff68377ebbbc2dcf3885dad326de4908a))
[15:50:50]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 1 commit to typing: nginxparser: ignore its typing ([c3a304e1](https://github.com/YunoHost/package_linter/commit/c3a304e1f4d43840531be00759c2fcae2c07b7d3))
[15:54:45]
<Yunohost Git/Infra notifications> [apps] mdtita [commented](https://github.com/YunoHost/apps/pull/2706#issuecomment-2480627616) on [issue #2706](https://github.com/YunoHost/apps/pull/2706) headplane_ynh: Done
[16:03:44]
<Yunohost Git/Infra notifications> [apps] tituspijean [commented](https://github.com/YunoHost/apps/pull/2706#discussion_r1845019552) on pull request #2706 headplane_ynh: suggestion url = "https://github.com/YunoHost-Apps/headplane_ynh"
[16:42:18]
<Yunohost Git/Infra notifications> [package_linter] Salamandar created new branch pathlib
[16:42:18]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 1 commit to pathlib: Use pathlib ([df1b2125](https://github.com/YunoHost/package_linter/commit/df1b2125d89eaf4b6700daa686227be457a9975c))
[17:23:31]
<Yunohost Git/Infra notifications> [apps] ericgaspar approved [pull request #2706](https://github.com/YunoHost/apps/pull/2706#pullrequestreview-2440775764) headplane_ynh
[17:23:41]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #2706](https://github.com/YunoHost/apps/pull/2706): headplane_ynh
[17:23:42]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to master: headplane_ynh (#2706) * headplane_ynh * clean up comments * order fix * change url * Update apps.toml * Update app... ([2b859364](https://github.com/YunoHost/apps/commit/2b859364f5c8f22587b81360e4ac3a4e41024e30))
[17:24:19]
<Yunohost Git/Infra notifications> [apps] yunohost-bot pushed 1 commit to master: Automatically add dates to the catalog files ([71272c65](https://github.com/YunoHost/apps/commit/71272c6518234145c329775629d1c433929dd5ae))
[18:13:50]
<eric_G> I feel like we should push https://github.com/YunoHost-Apps/nextcloud_ynh/pull/748 to master
[18:15:04]
<eric_G> and fix the timeout issue in the CI for Nextcloud...
[18:39:44]
<thatoo> Hello, I'd like to replace
`` $this->updateEmail($ldapEntry[$attr][0]); ``
to
`` $this->updateEmail($ldapEntry[$attr][1]); ``
using
`` ynh_replace_string --match_string="match_string" --replace_string="replace_string" --target_file="$install_dir/some_config_file" ``
but because of variables, I'm not sure how to do that.
[23:44:47]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u approved [pull request #748](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/748#pullrequestreview-2440825623) Testing
[23:47:21]
<kayou> IO on barbecue are too slow for all CIs :/
[23:50:45]
<kayou> You can escape `$` char in match and replace strings like that `\$`. But I feel like you have to escape `( ) [ ]` too
[23:51:04]
<kayou> Maybe not in replace string :/
[23:51:59]
<kayou> Well you can test by sourcing the helper in terminal, and call the function