[10:05:51]
<Yunohost Git/Infra notifications> [issues] xplosionmind [commented](https://github.com/YunoHost/issues/issues/1445#issuecomment-2132161560) on [issue #1445](https://github.com/YunoHost/issues/issues/1445) The website is outdated and doesnt properly explain what Yunohost does and why people could be interested: Hi I finally managed to find some time to dedicate to this task, so that it can be addressed properly. [I wrote a post...
[13:10:24]
<ChriChri> Just to share a feeling 🥴: I am about to test re-designed versions of [`ynh_local_curl`](https://codeberg.org/flohmarkt/flohmarkt_ynh/src/commit/82725cc58bf1751245c8820d4d5172091e673104/scripts/_common.sh#L310) and [`ynh_handle_getopts_args`](https://codeberg.org/flohmarkt/flohmarkt_ynh/src/commit/82725cc58bf1751245c8820d4d5172091e673104/scripts/_common.sh#L55) inside the flohmarkt scripts
[13:10:48]
<selfhoster1312> 🥴 is a good summary of bash feeling, good luck <3
[13:12:21]
<selfhoster1312> .:w
[13:14:41]
<ChriChri> bash should be renamed to pfash: pitfalls advanced shell
[13:19:17]
<selfhoster1312> broken again shell? :)
[13:29:32]
<Salamandar> @chrichri:librem.one what's the tl;dr of the changes ?
[13:29:48]
<Salamandar> i'm going to start working on helpers v2.1, maybe your input could be interesting
[13:42:11]
<ChriChri> > <@Salamandar:matrix.org> @chrichri:librem.one what's the tl;dr of the changes ?
I tried to update the headers as well. You can read it there. Short: getopts_args supports positional parameters instead of 'legacy' and allows to combine them with named parameters (long or short version). local_curl offers a bunch of new parameters including put and post and a not yet filled parameter (just remembered writing this😏) to check the result.
[14:11:07]
<Salamandar> Alright. You could do a pr on the official helpers too for discussion :)
[14:24:37]
<Yunohost Git/Infra notifications> [yunohost] tituspijean pushed 1 commit to mailalias-force: Allow to force-adding aliases for special groups ([1e13c618](https://github.com/YunoHost/yunohost/commit/1e13c618cf6ee47a4ef16b4ba4d47ca9a57a69c1))
[14:24:37]
<Yunohost Git/Infra notifications> [yunohost] tituspijean created new branch mailalias-force
[14:24:48]
<ChriChri> > <@Salamandar:matrix.org> Alright. You could do a pr on the official helpers too for discussion :)
will do when the missing option is filled 👍
[14:27:55]
<Yunohost Git/Infra notifications> [yunohost] tituspijean opened [pull request #1850](https://github.com/YunoHost/yunohost/pull/1850): Allow to force-adding aliases for special groups
[16:02:08]
<Yunohost Git/Infra notifications> [pepettes] OniriCorpe opened [issue #11](https://github.com/YunoHost/pepettes/issues/11): Clarifying who receives donations
[16:19:20]
<Yunohost Git/Infra notifications> [pepettes] OniriCorpe assigned OniriCorpe on [issue #11](https://github.com/YunoHost/pepettes/issues/11): Clarifying who receives donations
[16:19:20]
<Yunohost Git/Infra notifications> [pepettes] OniriCorpe labeled enhancement on [issue #11](https://github.com/YunoHost/pepettes/issues/11): Clarifying who receives donations
[17:22:00]
<Yunohost Git/Infra notifications> [yunohost] Salamandar created new branch helpers
[17:22:03]
<Yunohost Git/Infra notifications> [yunohost] Salamandar pushed 1 commit to helpers: Create versionned directories of the helpers ([97d30c50](https://github.com/YunoHost/yunohost/commit/97d30c5097111e4b204f0088a7549c32122b1f4c))
[17:26:38]
<Yunohost Git/Infra notifications> [yunohost] Salamandar [commented](https://github.com/YunoHost/yunohost/pull/1717#issuecomment-2132292948) on [issue #1717](https://github.com/YunoHost/yunohost/pull/1717) Create versionned directories of the helpers: > Im not sure that the package version must follow the helper version. To me its completely uncorrelated, and we could...
[17:28:36]
<Yunohost Git/Infra notifications> [yunohost] Salamandar [commented](https://github.com/YunoHost/yunohost/pull/1717#issuecomment-2132292948) on [issue #1717](https://github.com/YunoHost/yunohost/pull/1717) Create versionned directories of the helpers: > Im not sure that the package version must follow the helper version. To me its completely uncorrelated, and we could...
[17:46:36]
<Yunohost Git/Infra notifications> [yunohost] Salamandar pushed 2 commits to helpers ([97d30c509711...251b3f9324be](https://github.com/YunoHost/yunohost/compare/97d30c509711...251b3f9324be))
[18:04:30]
<Yunohost Git/Infra notifications> [issues] alexAubin opened [issue #2403](https://github.com/YunoHost/issues/issues/2403): Helpers 2.1 wishlist
[18:04:30]
<Yunohost Git/Infra notifications> [issues] alexAubin labeled :birthday: feature on [issue #2403](https://github.com/YunoHost/issues/issues/2403): Helpers 2.1 wishlist
[18:10:57]
<Yunohost Git/Infra notifications> [yunohost] alexAubin created new branch add-jinja-support-to-ynh-add-config
[18:10:57]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 1 commit to add-jinja-support-to-ynh-add-config: helpers: Add a --jinja option to ynh_add_config ([12764652](https://github.com/YunoHost/yunohost/commit/12764652b0469ba4d839ec80c3481b11d3546842))
[18:12:57]
<Yunohost Git/Infra notifications> [yunohost] alexAubin opened [pull request #1851](https://github.com/YunoHost/yunohost/pull/1851): helpers: Add a --jinja option to ynh_add_config
[18:54:14]
<selfhoster1312> \o/
[18:56:07]
<selfhoster1312> Aleks (he/him/il/lui), what would you think about adding Jinja templates for specific hooks in the manifest.toml so we don't even have to go through bash for that?
[18:57:44]
<selfhoster1312> maybe something like
```
[resources.templates.nginx]
source = "conf/nginx.conf"
dest = "/etc/nginx/conf.d/{{ app_id }}.conf"
hooks = [ "install", "upgrade" ]
```
[18:57:51]
<selfhoster1312> manifest v3 wishlist? :P
[18:58:52]
<selfhoster1312> we could also have a jinja expression as conditional based on app settings
[18:59:22]
<Yunohost Git/Infra notifications> [issues] FranzMari opened [issue #2404](https://github.com/YunoHost/issues/issues/2404): Edge case in port availability check
[18:59:22]
<Yunohost Git/Infra notifications> [issues] FranzMari labeled :space_invader: bug on [issue #2404](https://github.com/YunoHost/issues/issues/2404): Edge case in port availability check
[18:59:49]
<selfhoster1312> i'm not a big fan, but ansible does stuff like that:
```
when = "{{ app.settings.nginx_enabled }}"
```
[19:42:01]
<Aleks (he/him/il/lui)> > <selfhoster1312> manifest v3 wishlist? :P
It's already in there