[00:18:31]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to master: move rainloop to graveyard (#2317) Co-authored-by: OniriCorpe <OniriCorpe@users.noreply.github.com> ([1638cc7c](https://github.com/YunoHost/apps/commit/1638cc7c54e7a9f7ab0e390726e6c345a9a166df))
[00:18:31]
<Yunohost Git/Infra notifications> [apps] alexAubin merged [pull request #2317](https://github.com/YunoHost/apps/pull/2317): move rainloop to graveyard
[00:18:32]
<Yunohost Git/Infra notifications> [apps] alexAubin deleted branch rainloop-graveyard
[00:18:52]
<Yunohost Git/Infra notifications> [apps] yunohost-bot pushed 1 commit to master: Automatically add dates to the catalog files ([81a4fdbf](https://github.com/YunoHost/apps/commit/81a4fdbf69c9e21e688778516acc3b20bb5d00c8))
[03:52:07]
<Yunohost Git/Infra notifications> App endi failed all tests in job [#26299](https://ci-apps.yunohost.org/ci/job/26299) :(
[07:02:03]
<Yunohost Git/Infra notifications> [apps] yunohost-bot labeled Wishlist on [pull request #2318](https://github.com/YunoHost/apps/pull/2318): Add Music Assistant to wishlist
[07:02:03]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2318](https://github.com/YunoHost/apps/pull/2318): Add Music Assistant to wishlist
[07:02:03]
<Yunohost Git/Infra notifications> [apps] yunohost-bot created new branch add-to-wishlist-music-assistant
[07:02:03]
<Yunohost Git/Infra notifications> [apps] yunohost-bot pushed 1 commit to add-to-wishlist-music-assistant: Add Music Assistant to wishlist ([434bd6c8](https://github.com/YunoHost/apps/commit/434bd6c8b9b0564f396cfd2b1864b2f61e27111f))
[08:00:20]
<eric_G> I am wondering about this autopatch `TEST BEFORE MERGE ynh_setup_source --full_replace=1` that I see in every repo. Is it wise to apply this in every repo? if so why not include full_replace in the core?
[08:00:24]
<ChriChri> The README* files are generated automatically so the files differ from the local repo. Is it a good idea to have a .gitignore to ignore these files?
[08:37:33]
<orhtej2> > <@ericg:matrix.org> I am wondering about this autopatch `TEST BEFORE MERGE ynh_setup_source --full_replace=1` that I see in every repo. Is it wise to apply this in every repo? if so why not include full_replace in the core?
It would break apps that don't restore config so probably not an universal fix
[08:56:52]
<ChriChri> eric_G: Did you intentionally (re)start a `!testme` for flohmarkt_ynh ? I just was about to delete the branch `testing` to replace it by one with some more debugging (to get rid of the warnings and errors still contained in the CI output).
[09:07:09]
<eric_G> ChriChri: Yes, I pushed some fixes to remove warnings.
[09:07:25]
<ChriChri> Ah, I'll restart it in a minute :)
[09:09:09]
<eric_G> and no need to remove the branch all together, just add to it
[09:09:43]
<ChriChri> I'm avoiding the pull/rebase push cycle (README*)...
[09:10:53]
<eric_G> > <@chrichri:librem.one> I'm avoiding the pull/rebase push cycle (README*)...
just merge master to testing et voilà
[09:14:39]
<eric_G> I still don't understand why you are closing the testing branch 🤔
[09:16:08]
<ChriChri> no git professional on this end of the line - didn't want to look into how to ignore README* and the fact that I did not have the version from master in my local repo.
[09:34:40]
<Yunohost Git/Infra notifications> Job [#26298](https://ci-apps.yunohost.org/ci/job/26298) for zeroui failed miserably :(
[09:38:41]
<Yunohost Git/Infra notifications> Job [#26303](https://ci-apps.yunohost.org/ci/job/26303) for itflow failed miserably :(
[09:51:02]
<ChriChri> > <@ericg:matrix.org> just merge master to testing et voilà
...and look out that there might be files missing in master (DISCLAIMER.md, fixed).
BTW: DISCLAIMER.md contains general information, but still leads to a linter warning. Should it be removed alltogether? It is stuff I'd like to have included into README.md...
[10:25:47]
<eric_G> I had `DISCLAIMER.md` removed because it is not supported by v2 apps. (that is why there is a warning in the CI)
[10:26:52]
<eric_G> also `DEVELOPMENT.md` is not needed as it concerns upstream
[10:29:55]
<eric_G> `PRE_INSTALL.md` should be concise as should `DESCRIPTION.md` be.
[10:53:53]
<ChriChri> Hm, on the catalog page of the app only information from README.md of upstream is presented?
Ah, just found that even POST_INSTALL popped up there now - is that new? ~~So I guess DISCLAIMER could go into PRE_INSTALL?~~ PRE_INSTALL doesn't seem to be presented there...
[10:56:47]
<ChriChri> Ah, o.k. now I understand what you did eric_G.
[11:01:55]
<Yunohost Git/Infra notifications> Job [#26302](https://ci-apps.yunohost.org/ci/job/26302) for matrix-appservice-irc failed miserably :(
[11:05:19]
<Yunohost Git/Infra notifications> Job [#26307](https://ci-apps.yunohost.org/ci/job/26307) for streams failed miserably :(
[11:07:42]
<Yunohost Git/Infra notifications> Job [#26308](https://ci-apps.yunohost.org/ci/job/26308) for terraforming-mars failed miserably :(
[11:09:58]
<Aleks (he/him/il/lui)> (lxd being lxd again)
[11:10:14]
<Yunohost Git/Infra notifications> Job [#26309](https://ci-apps.yunohost.org/ci/job/26309) for actual failed miserably :(
[11:16:00]
<ChriChri> eric_GThanks - guess I got rid of one linter warning.
…
Whooops, none linter warning left? No warning about `ynh_install_extra_app_dependencies` anymore? Is that accepted again?
[11:17:37]
<Yunohost Git/Infra notifications> Job [#26303](https://ci-apps.yunohost.org/ci/job/26303) for itflow failed miserably :(
[12:35:45]
<ChriChri> Until now I can't reproduce the warnings leading to `Error: There's quite a lot of warnings in the output ! If those warnings are coming from some app build step and ain't actual warnings, please redirect them to the standard output instead of the error output ...!` on my test system. https://ci-apps-bookworm-dev.yunohost.org/ci/job/855
[12:57:37]
<orhtej2> > <@chrichri:librem.one> Until now I can't reproduce the warnings leading to `Error: There's quite a lot of warnings in the output ! If those warnings are coming from some app build step and ain't actual warnings, please redirect them to the standard output instead of the error output ...!` on my test system. https://ci-apps-bookworm-dev.yunohost.org/ci/job/855
Its because it failed waiting on service restart i think
[14:11:35]
<ChriChri> Hm, `restart`? I just `stop` and `start` - does it get restarted internally by yunohost?
[15:02:41]
<ChriChri> I already know this one: `systemd[1]: flohmarkt_sub.domain.tld.service: Main process exited, code=exited, status=209/STDOUT`
Can't write to STDOUT (probably because logfile not writable).
[15:06:53]
<ChriChri> > Its because it failed waiting on service restart i think
I'd love to understand why I can't reproduce those WARNINGs on my test system.
[15:14:25]
<ChriChri> > <@chrichri:librem.one> I'd love to understand why I can't reproduce those WARNINGs on my test system.
Hm, for some reason there's already a flohmarkt up and running when the script tries to start it...
[15:59:51]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2319](https://github.com/YunoHost/apps/pull/2319): Translations update from Weblate for readme-generator
[16:00:38]
<Yunohost Git/Infra notifications> [apps] OniriCorpe pushed 1 commit to master: Translated readme-generator using Weblate (Spanish) Currently translated at 97.4% (38 of 39 strings) Translation: Yuno... ([0aa51844](https://github.com/YunoHost/apps/commit/0aa518443f65cca831ee3a0022c90b1d0b77e862))
[16:00:39]
<Yunohost Git/Infra notifications> [apps] OniriCorpe merged [pull request #2319](https://github.com/YunoHost/apps/pull/2319): Translations update from Weblate for readme-generator
[16:21:14]
<Yunohost Git/Infra notifications> App ztncui failed all tests in job [#26146](https://ci-apps.yunohost.org/ci/job/26146) :(
[17:00:52]
<Yunohost Git/Infra notifications> [apps] yunohost-bot created new branch update_app_levels
[17:00:52]
<Yunohost Git/Infra notifications> [apps] yunohost-bot pushed 1 commit to update_app_levels: Update app levels according to CI results ([f0093bb9](https://github.com/YunoHost/apps/commit/f0093bb96b1cf07873945711c7d0d17696049118))
[17:00:52]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2320](https://github.com/YunoHost/apps/pull/2320): Update app levels according to CI results
[17:18:08]
<Yunohost Git/Infra notifications> Job [#26302](https://ci-apps.yunohost.org/ci/job/26302) for matrix-appservice-irc failed miserably :(
[17:22:12]
<Yunohost Git/Infra notifications> Job [#26308](https://ci-apps.yunohost.org/ci/job/26308) for terraforming-mars failed miserably :(
[17:24:16]
<Yunohost Git/Infra notifications> Job [#26309](https://ci-apps.yunohost.org/ci/job/26309) for actual failed miserably :(
[17:26:43]
<Yunohost Git/Infra notifications> Job [#26311](https://ci-apps.yunohost.org/ci/job/26311) for ampache failed miserably :(
[17:29:19]
<Yunohost Git/Infra notifications> Job [#26312](https://ci-apps.yunohost.org/ci/job/26312) for concrete5 failed miserably :(
[17:31:38]
<Yunohost Git/Infra notifications> Job [#26313](https://ci-apps.yunohost.org/ci/job/26313) for drupal failed miserably :(
[17:36:28]
<Yunohost Git/Infra notifications> Job [#26315](https://ci-apps.yunohost.org/ci/job/26315) for grr failed miserably :(
[17:38:58]
<Yunohost Git/Infra notifications> Job [#26316](https://ci-apps.yunohost.org/ci/job/26316) for homarr failed miserably :(
[17:41:17]
<Yunohost Git/Infra notifications> Job [#26317](https://ci-apps.yunohost.org/ci/job/26317) for jellyfin-vue failed miserably :(
[17:43:43]
<Yunohost Git/Infra notifications> Job [#26318](https://ci-apps.yunohost.org/ci/job/26318) for syncthing failed miserably :(
[17:46:06]
<Yunohost Git/Infra notifications> Job [#26319](https://ci-apps.yunohost.org/ci/job/26319) for z-push failed miserably :(
[17:51:26]
<Émy - OniriCorpe> ci-apps fixed
[18:06:05]
<ChriChri> Uhm, maybe I misunderstood something about the yunohost_app repositories. An unfinished testing branch just got merged on the repo I'm working on by eric_G. Now the master branch is published with a version that doesn't have a working backup/restore, yet.
I thought that if I take the responsibility for such a repository I'd be the one at least to be consulted before someone else does changes? Isn't that meant to be like that?
The testing branch I'm writing about had only been uploaded to run on the CI and was never meant for production.
Probably I didn't understand the whole concept...
[18:07:45]
<eric_G> Sorry my fault then... Maybe I should leave your repo alone 😅
[18:10:46]
<ChriChri> eric_GNo, please not! I like very much if there's someone correcting my mistakes and misunderstandings and maybe I didn't understand each option of communication through github completely, yet.
It's nice not being alone on this. But no matter how we got here we need to improve our cooperation for the future?
Is there an easy way to roleback the merge or should I better upgrade by another merge to stop people from installing more than one flohmarkt instance on the same yunohost (which is still the problem for backup/restore)?
[18:13:35]
<ChriChri> As I'm still quite new on yunohost I need any help I can get... and I can't write often enough **thank you** here on the channel. The more I read about yunohost the more respect I have about the **gigantic task** the community and especially the people mostly active here are acomplishing.
[18:20:07]
<ChriChri> I guess it is back to ynh4 now which it seems I never published...
[18:22:44]
<ChriChri> O.k. it is `multi_instance = false` again...
[18:29:22]
<ChriChri> I guess I'll need to look into the configuration how to accept merges only with a certain number of Reviews - that would be the correct way, right?
[18:53:12]
<eric_G> or set your PR to draft until you finish working on it...
[19:06:59]
<ChriChri> > <@ericg:matrix.org> or set your PR to draft until you finish working on it...
OK, need to look how to do that...
[19:54:33]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 4 commits to update_app_levels ([f0093bb96b1c...d8f7cecd29b8](https://github.com/YunoHost/apps/compare/f0093bb96b1c...d8f7cecd29b8))
[19:54:45]
<Yunohost Git/Infra notifications> [apps] github-actions[bot] created new branch actions/toml
[19:54:46]
<Yunohost Git/Infra notifications> [apps] github-actions[bot] pushed 1 commit to actions/toml: :art: Format TOML with Taplo ([d064a423](https://github.com/YunoHost/apps/commit/d064a423223cd6625a91ca84ac8057bab9ef142e))
[19:54:46]
<Yunohost Git/Infra notifications> [apps] github-actions[bot] opened [pull request #2321](https://github.com/YunoHost/apps/pull/2321): Format TOML with Taplo
[19:55:12]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([8973545b](https://github.com/YunoHost/apps/commit/8973545b977bd8e36e2db53ccabe364723408a37))
[19:55:15]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2320](https://github.com/YunoHost/apps/pull/2320): Update app levels according to CI results
[19:55:27]
<Yunohost Git/Infra notifications> [apps] github-actions[bot] pushed 1 commit to actions/toml: :art: Format TOML with Taplo ([a3515938](https://github.com/YunoHost/apps/commit/a3515938e2c91bd5262e222e375f1e42654235c0))
[19:55:38]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([defd8061](https://github.com/YunoHost/apps/commit/defd8061137322466d8b2f74c225002b42ac3240))
[19:55:40]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2320](https://github.com/YunoHost/apps/pull/2320): Update app levels according to CI results
[19:55:52]
<Yunohost Git/Infra notifications> [apps] github-actions[bot] pushed 1 commit to actions/toml: :art: Format TOML with Taplo ([32774760](https://github.com/YunoHost/apps/commit/32774760b27dece2e88ef7e713a48b3993f3d8b6))
[19:56:09]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([0e9de04b](https://github.com/YunoHost/apps/commit/0e9de04b22ba3dac4fdeac21a8e62a189a098a78))
[19:56:09]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2320](https://github.com/YunoHost/apps/pull/2320): Update app levels according to CI results
[19:56:21]
<Yunohost Git/Infra notifications> [apps] github-actions[bot] pushed 1 commit to actions/toml: :art: Format TOML with Taplo ([f5493064](https://github.com/YunoHost/apps/commit/f54930642e95b29717370715b46ace18f0d75182))
[19:57:21]
<ChriChri> > <@ericg:matrix.org> or set your PR to draft until you finish working on it...
Hey eric_G - read about the draft feature and set the PR to draft - sorry for the confusion! Please have a look whether I did it right.
[19:57:52]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([5092f396](https://github.com/YunoHost/apps/commit/5092f396858d24d62d82bf4c812de4a7606afae2))
[19:57:57]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2320](https://github.com/YunoHost/apps/pull/2320): Update app levels according to CI results
[19:58:05]
<Yunohost Git/Infra notifications> [apps] github-actions[bot] pushed 1 commit to actions/toml: :art: Format TOML with Taplo ([6a134e3e](https://github.com/YunoHost/apps/commit/6a134e3e4dd530b411a7a0e925ae8c2144145e2f))
[19:59:21]
<Yunohost Git/Infra notifications> [apps] ericgaspar approved [pull request #2320](https://github.com/YunoHost/apps/pull/2320#pullrequestreview-2050782948) Update app levels according to CI results
[20:12:54]
<Yunohost Git/Infra notifications> [apps] OniriCorpe pushed 1 commit to update_app_levels: remove duplicated level key for flohmarkt ([cdd26131](https://github.com/YunoHost/apps/commit/cdd261318f8d7addf0cd626622a871d59d9e03b2))
[20:14:58]
<Yunohost Git/Infra notifications> [apps] OniriCorpe edited [pull request #2321](https://github.com/YunoHost/apps/pull/2321): Format TOML with Taplo
[20:14:58]
<Yunohost Git/Infra notifications> [apps] OniriCorpe pushed 9 commits to master ([0aa518443f65...74fa9dc137df](https://github.com/YunoHost/apps/compare/0aa518443f65...74fa9dc137df))
[20:14:58]
<Yunohost Git/Infra notifications> [apps] OniriCorpe merged [pull request #2320](https://github.com/YunoHost/apps/pull/2320): Update app levels according to CI results
[20:14:59]
<Yunohost Git/Infra notifications> [apps] OniriCorpe deleted branch update_app_levels
[20:16:13]
<Yunohost Git/Infra notifications> [apps] OniriCorpe pushed 10 commits to actions/toml ([6a134e3e4dd5...8c737d3db051](https://github.com/YunoHost/apps/compare/6a134e3e4dd5...8c737d3db051))
[20:16:19]
<Yunohost Git/Infra notifications> [apps/actions/toml] Update apps.toml - Éric Gaspar
[20:16:32]
<Yunohost Git/Infra notifications> [apps] OniriCorpe closed [pull request #2321](https://github.com/YunoHost/apps/pull/2321): Format TOML with Taplo
[20:16:34]
<Yunohost Git/Infra notifications> [apps] OniriCorpe deleted branch actions/toml