[07:35:14]
<Yunohost Git/Infra notifications> [yunohost] Salamandar pushed 1 commit to fix_ci: Potential fix for code scanning alert no. 693: Potentially uninitialized local variable Co-authored-by: Copilot Autofix... ([917d3ccd](https://github.com/YunoHost/yunohost/commit/917d3ccdfbd458c73536304c8c81ee74d4674041))
[10:41:21]
<Yunohost Git/Infra notifications> [yunohost] fflorent edited [pull request #2122](https://github.com/YunoHost/yunohost/pull/2122): Implement RedisAppResource
[10:45:31]
<Yunohost Git/Infra notifications> [yunohost] fflorent opened [pull request #2124](https://github.com/YunoHost/yunohost/pull/2124): Redis app resource (alternate implementation)
[10:45:45]
<Yunohost Git/Infra notifications> [yunohost] fflorent edited [pull request #2122](https://github.com/YunoHost/yunohost/pull/2122): Implement RedisAppResource
[10:48:08]
<Yunohost Git/Infra notifications> [yunohost] fflorent [commented](https://github.com/YunoHost/yunohost/pull/2122#discussion_r2147524963) on pull request #2122 Implement RedisAppResource: I am not very fund of provisionned_redis_db_names, I feel that it is a bit awkward. But it sounds like a viable hack. ...
[10:51:28]
<Yunohost Git/Infra notifications> [yunohost] fflorent edited [pull request #2124](https://github.com/YunoHost/yunohost/pull/2124): Redis app resource (alternate implementation)
[10:56:51]
<florent[m]> Aleks (he/him/il/lui): my work is ready to be reviewed.
You can pick either one of these PRs, I'll clean the mess as soon as you tell me which approach you prefer:
- with some kind of viable hack (storing the previous databases in a setting): https://github.com/YunoHost/yunohost/pull/2122
- with a proposal to get rid of the previous hack: https://github.com/YunoHost/yunohost/pull/2124
Of course, I welcome better alternatives to specify redis databases to provision in the manifest.
[13:55:57]
<Yunohost Git/Infra notifications> [yunohost] orhtej2 just made [pull request #2118](https://github.com/YunoHost/yunohost/pull/2118) ready for review: Add support for (plain) XZ and ZST source archives
[13:55:58]
<Yunohost Git/Infra notifications> [yunohost] orhtej2 [commented](https://github.com/YunoHost/yunohost/pull/2118#issuecomment-2973887430) on [issue #2118](https://github.com/YunoHost/yunohost/pull/2118) Add support for (plain) XZ and ZST source archives: The results are in: - tar.xz - [test app](https://github.com/orhtej2/compression_ynh/tree/tar_xz), [log](https://paste.y...
[16:20:56]
<Yunohost Git/Infra notifications> [yunohost] fflorent edited a [comment](https://github.com/YunoHost/yunohost/pull/2122#discussion_r2147524963) on pull request #2122 Implement RedisAppResource: I am not very fond of provisionned_redis_db_names, I feel that it is a bit awkward. But it sounds like a viable hack. ...
[17:18:06]
<Yunohost Git/Infra notifications> [yunohost] fflorent [commented](https://github.com/YunoHost/yunohost/pull/2122#discussion_r2148564977) on pull request #2122 Implement RedisAppResource: suggestion app_name = self.app.replace("-", "_").replace(".", "_") os.system(fredis-...
[17:18:26]
<Yunohost Git/Infra notifications> [yunohost] fflorent edited a [comment](https://github.com/YunoHost/yunohost/pull/2122#discussion_r2148564977) on pull request #2122 Implement RedisAppResource: Probably more interesting for debugging to set the value to the app name. suggestion app_name = self...