[11:43:06]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 5 commits to main ([ca58c03355ad...7fd4ca0f6115](https://github.com/YunoHost/cli/compare/ca58c03355ad...7fd4ca0f6115))
[11:43:06]
<Yunohost Git/Infra notifications> [cli/main] main: Set SSE log handler - Félix Piédallu
[11:43:06]
<Yunohost Git/Infra notifications> [cli/main] Remove dependency on colored - Félix Piédallu
[11:43:07]
<Yunohost Git/Infra notifications> [cli/main] Fix typing - Félix Piédallu
[11:46:36]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 1 commit to main: Fix issues reported by linters ([37ec0703](https://github.com/YunoHost/cli/commit/37ec070395d43fa930076fe72097695a12a01c06))
[11:51:30]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 2 commits to main ([37ec070395d4...231a63793800](https://github.com/YunoHost/cli/compare/37ec070395d4...231a63793800))
[11:51:34]
<Yunohost Git/Infra notifications> [cli/main] Add support for pretty human data printing - Félix Piédallu
[11:51:35]
<Yunohost Git/Infra notifications> [cli/main] isort, ruff - Félix Piédallu
[11:54:52]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 1 commit to main: interactive: write dummy ask_until_valid ([52bc072b](https://github.com/YunoHost/cli/commit/52bc072b2938a632851a389608788b67280cec58))
[12:10:34]
<Salamandar> https://aria.im/_bifrost/v1/media/download/AeuSSylFkayGSMpKfLMoWBTEq461Ms6aF-Hmi-7K59tQyGiMskojnNbBk6KrapJgn1SmQtdvUrycucVPb8w2VsFCeXc3-lwQAG1hdHJpeC5vcmcvY3lpdGJjTG9JZWpodnhZaHRrUlBpSlNL
[12:10:41]
<Salamandar> daily bragging about the future tool
[12:21:58]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 2 commits to main ([f3101bcb2986...1d424548dbbe](https://github.com/YunoHost/cli/compare/f3101bcb2986...1d424548dbbe))
[12:21:59]
<Yunohost Git/Infra notifications> [cli/main] Update human data output - Félix Piédallu
[12:21:59]
<Yunohost Git/Infra notifications> [cli/main] Fix SSE event parsing - Félix Piédallu
[12:43:08]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 4 commits to main ([a865fd2b07dd...28d1bc71b3bd](https://github.com/YunoHost/cli/compare/a865fd2b07dd...28d1bc71b3bd))
[12:43:08]
<Yunohost Git/Infra notifications> [cli/main] repr_simple now allows integers - Félix Piédallu
[12:43:09]
<Yunohost Git/Infra notifications> [cli/main] human: support settings list - Félix Piédallu
[12:43:09]
<Yunohost Git/Infra notifications> [cli/main] Fix human printing - Félix Piédallu
[13:29:31]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 1 commit to main: repr_simple now allows float ([d1b0d6a3](https://github.com/YunoHost/cli/commit/d1b0d6a3cb746bcfd566a8ce9960e67d7fd4d327))
[14:18:31]
<Yunohost Git/Infra notifications> [yunohost] Salamandar pushed 1 commit to pyproject: Replace tox.ini, pytest.ini, .coveragerc with pyproject.toml flake8 maintainers dont want to hear about pyproject.toml ([3b9ebfe3](https://github.com/YunoHost/yunohost/commit/3b9ebfe3599426c0d1aea3d4359c9aa5958ac5ec))
[14:18:31]
<Yunohost Git/Infra notifications> [yunohost] Salamandar created new branch pyproject
[14:18:33]
<Yunohost Git/Infra notifications> [yunohost] Salamandar opened [pull request #2120](https://github.com/YunoHost/yunohost/pull/2120): Use a pyproject.toml file instead of all the other files.
[14:18:34]
<Yunohost Git/Infra notifications> [yunohost] Salamandar assigned Salamandar on [pull request #2120](https://github.com/YunoHost/yunohost/pull/2120): Use a pyproject.toml file instead of all the other files.
[14:18:45]
<Yunohost Git/Infra notifications> [yunohost] Salamandar [commented](https://github.com/YunoHost/yunohost/pull/2120#issuecomment-2972789201) on [issue #2120](https://github.com/YunoHost/yunohost/pull/2120) Use a pyproject.toml file instead of all the other files.: Please wait for CI to run before merging.
[17:29:22]
<Yunohost Git/Infra notifications> [yunohost/pyproject] Replace tox.ini, pytest.ini, .coveragerc with pyproject.toml flake8 maintainers dont want to hear about pyproject.toml - Félix Piédallu
[17:29:22]
<Yunohost Git/Infra notifications> [yunohost/pyproject] Ensure debhelper uses distutil pybuild system - Félix Piédallu
[17:29:22]
<Yunohost Git/Infra notifications> [yunohost] Salamandar pushed 2 commits to pyproject ([3b9ebfe35994...6eb7d22f4511](https://github.com/YunoHost/yunohost/compare/3b9ebfe35994...6eb7d22f4511))
[18:44:06]
<Yunohost Git/Infra notifications> [moulinette] Salamandar closed [pull request #366](https://github.com/YunoHost/moulinette/pull/366): Use pyproject.toml instead of setup.py
[18:44:06]
<Yunohost Git/Infra notifications> [moulinette] Salamandar deleted branch pyproject
[19:10:24]
<Yunohost Git/Infra notifications> [yunohost] Salamandar created new branch relative_imports
[19:10:25]
<Yunohost Git/Infra notifications> [yunohost] Salamandar pushed 2 commits to relative_imports ([f4e1b2094275^...6e7314531cd7](https://github.com/YunoHost/yunohost/compare/f4e1b2094275^...6e7314531cd7))
[19:10:27]
<Yunohost Git/Infra notifications> [yunohost/relative_imports] src: Replace absolute yunohost.xx imports with relative .xx imports - Félix Piédallu
[19:10:28]
<Yunohost Git/Infra notifications> [yunohost/relative_imports] src: In subdirs, replace absolute yunohost.xx imports with relative ..xx imports - Félix Piédallu
[19:10:43]
<Yunohost Git/Infra notifications> [yunohost] Salamandar opened [pull request #2121](https://github.com/YunoHost/yunohost/pull/2121): src: Replace absolute yunohost.xx imports with relative .xx imports
[19:12:11]
<Yunohost Git/Infra notifications> [yunohost] github-advanced-security[bot] commented [pull request #2121](https://github.com/YunoHost/yunohost/pull/2121#pullrequestreview-2928663709) src: Replace absolute yunohost.xx imports with relative .xx imports: CodeQL found more than 20 potential problems in the proposed changes. Check the [Files changed](/YunoHost/yunohost/pull/...
[19:49:58]
<Yunohost Git/Infra notifications> [yunohost] Salamandar pushed 2 commits to pyproject ([2b02a63c93e9...ab67cf945b38](https://github.com/YunoHost/yunohost/compare/2b02a63c93e9...ab67cf945b38))
[19:50:00]
<Yunohost Git/Infra notifications> [yunohost/pyproject] Replace tox.ini, pytest.ini, .coveragerc with pyproject.toml flake8 maintainers dont want to hear about pyproject.toml - Félix Piédallu
[19:50:01]
<Yunohost Git/Infra notifications> [yunohost/pyproject] Add Python deps for uv/venv management - Félix Piédallu
[20:08:29]
<Yunohost Git/Infra notifications> [yunohost] fflorent opened [pull request #2122](https://github.com/YunoHost/yunohost/pull/2122): Implement RedisAppResource (WIP)
[20:12:10]
<Yunohost Git/Infra notifications> [yunohost] Salamandar pushed 3 commits to fix_ci ([6991eda2834c^...0c79f80aa0fe](https://github.com/YunoHost/yunohost/compare/6991eda2834c^...0c79f80aa0fe))
[20:12:10]
<Yunohost Git/Infra notifications> [yunohost] Salamandar created new branch fix_ci
[20:12:11]
<Yunohost Git/Infra notifications> [yunohost/fix_ci] Add missing translations - Félix Piédallu
[20:12:11]
<Yunohost Git/Infra notifications> [yunohost/fix_ci] Fix linter issues - Félix Piédallu
[20:12:12]
<Yunohost Git/Infra notifications> [yunohost/fix_ci] Fix mypy issues - Félix Piédallu
[20:12:28]
<Yunohost Git/Infra notifications> [yunohost] Salamandar opened [pull request #2123](https://github.com/YunoHost/yunohost/pull/2123): Fix CI issues
[20:13:08]
<Yunohost Git/Infra notifications> [yunohost] fflorent [commented](https://github.com/YunoHost/yunohost/pull/2122#discussion_r2147272519) on pull request #2122 Implement RedisAppResource (WIP): I need help for that part. The idea is that if the packager removes a Redis database (say [resources.redis.celery_red...
[20:13:57]
<Yunohost Git/Infra notifications> [issues] fflorent [commented](https://github.com/YunoHost/issues/issues/2583#issuecomment-2973136879) on [issue #2583](https://github.com/YunoHost/issues/issues/2583) package linter: Detect when restoration does not look up for a free Redis DB: @zamentur Thanks for the suggestion. Done in https://github.com/YunoHost/yunohost/pull/2122
[20:13:58]
<Yunohost Git/Infra notifications> [yunohost] github-advanced-security[bot] [commented](https://github.com/YunoHost/yunohost/pull/2123#discussion_r2147272695) on pull request #2123 Fix CI issues: ## Statement has no effect
This statement has no effect.
[Show more details](https://github.com/YunoHost/yunohost/secu...
[20:13:58]
<Yunohost Git/Infra notifications> [yunohost] github-advanced-security[bot] [commented](https://github.com/YunoHost/yunohost/pull/2123#discussion_r2147272690) on pull request #2123 Fix CI issues: ## Potentially uninitialized local variable
Local variable todos_create may be used before it is initialized.
[Show ...
[20:13:59]
<Yunohost Git/Infra notifications> [yunohost] github-advanced-security[bot] [commented](https://github.com/YunoHost/yunohost/pull/2123#discussion_r2147272696) on pull request #2123 Fix CI issues: ## Statement has no effect
This statement has no effect.
[Show more details](https://github.com/YunoHost/yunohost/secu...
[20:13:59]
<Yunohost Git/Infra notifications> [yunohost] github-advanced-security[bot] [commented](https://github.com/YunoHost/yunohost/pull/2123#discussion_r2147272691) on pull request #2123 Fix CI issues: ## Potentially uninitialized local variable
Local variable todos_update may be used before it is initialized.
[Show ...
[20:13:59]
<Yunohost Git/Infra notifications> [yunohost] github-advanced-security[bot] [commented](https://github.com/YunoHost/yunohost/pull/2123#discussion_r2147272693) on pull request #2123 Fix CI issues: ## Potentially uninitialized local variable
Local variable todos_update may be used before it is initialized.
[Show ...
[20:14:00]
<Yunohost Git/Infra notifications> [yunohost] github-advanced-security[bot] [commented](https://github.com/YunoHost/yunohost/pull/2123#discussion_r2147272692) on pull request #2123 Fix CI issues: ## Potentially uninitialized local variable
Local variable todos_create may be used before it is initialized.
[Show ...
[20:16:08]
<Yunohost Git/Infra notifications> [yunohost] fflorent edited [pull request #2122](https://github.com/YunoHost/yunohost/pull/2122): Implement RedisAppResource (WIP)
[20:18:28]
<florent[m]> Aleks (he/him/il/lui): Quick question for my PR above. I see that the `packaging-v3` branch exists. Should my PR be merged in this branch or to the `dev` one?
[20:19:42]
<florent[m]> (otherwise, I would need help to understand how to test it and my comments, not that's not the so-quick questions, I let you answer these questions when you can :))
[20:21:55]
<Aleks (he/him/il/lui)> hmmmm yeah it depends on eeeeh the timescale we're aiming for for this new declarative redis thing 😅 I'm actually working right now in refactoring the resource system for several reason (though packaging v2 will be backward compatible), with one motivation being to allow for several database declarations (similar to the `permissions` or `ports` or ... where we have a "main" one and possibly others)
In particular relevant for redis because we could have a "redis" database type part of what's handled in the `[resources.database]` stuff with the consideration that apps that want to have a redis db also have an sql db i suppose
[20:22:27]
<Aleks (he/him/il/lui)> (dunno if that explanation makes sense)
[20:23:10]
<Aleks (he/him/il/lui)> but that kinda impact the way to implement the redis db handling 😬 (though it's not like it's "rewriting everything")
[20:24:34]
<Aleks (he/him/il/lui)> anyway if in the current state you want to write tests for the new resource you can have a look at the database/mysql resource test
[20:24:38]
<Aleks (he/him/il/lui)> here : https://github.com/YunoHost/yunohost/blob/dev/src/tests/test_app_resources.py#L294
[20:27:06]
<Aleks (he/him/il/lui)> during the test you'll also want to make sure that the appropriate redis package is installed on the maniche ... naively I would add a yolo `os.system('apt install redis-server -y')` in the `setup_function()` near the top of the file (mariadb-server-whatever is in fact installed somewhere else such that the mysql test can indeed run)
[20:28:33]
<Aleks (he/him/il/lui)> and the tests can be run in an ynh-dev env (do you have one somewhere somehow ?) using something like `./ynh-dev test yunohost:app_resources:resource_redis` (assuming you call your test function \`test\_resource\_redis`)
[20:28:41]
<Aleks (he/him/il/lui)> and the tests can be run in an ynh-dev env (do you have one somewhere somehow ?) using something like `./ynh-dev test yunohost:app_resources:resource_redis` (assuming you call your test function `test_resource_redis`)
[20:29:51]
<Yunohost Git/Infra notifications> [yunohost] fflorent edited [pull request #2122](https://github.com/YunoHost/yunohost/pull/2122): Implement RedisAppResource (WIP)
[20:29:53]
<Aleks (he/him/il/lui)> and of course(?) you can also test a "real" app install by adding the corresponding `[resources.rediswhatver]` in an app manifest locally and then `yunohost app install /path/to/the/app/folder` (in an ynh-dev setup as well)
[20:30:55]
<florent[m]> Was not obvious to me 😅
[20:31:21]
<florent[m]> Thanks for all the explanations, I will work on writing tests soon.
[20:58:53]
<florent[m]> I log out for now.
I have written tests, just need to run in a dedicated `ynh-dev` (I currently have one for packaging apps, so the environment is not clean).
Ideally I would like to have two containers, one for my apps, the other one for this kind of tests.
(I can RTFM that)
[20:58:58]
<florent[m]> Thanks for your help, very much appreciated!