[00:36:42]
<Yunohost Git/Infra notifications> App django-fmd rises from level 3 to 8 in job [#25769](https://ci-apps.yunohost.org/ci/job/25769) !
[00:42:50]
<Yunohost Git/Infra notifications> App grafana goes down from level 8 to 6 in job [#25520](https://ci-apps.yunohost.org/ci/job/25520)
[01:36:14]
<Yunohost Git/Infra notifications> App django_example goes down from level 8 to 3 in job [#25702](https://ci-apps.yunohost.org/ci/job/25702)
[02:12:16]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 74 pending update PRs
- 10 new apps PRs
- 5 failed apps updates: elasticsearch8, focalboard, owncloud, tvheadend, vikunja
See the full log here: http://paste.yunohost.org/raw/ikicoqatir
[02:34:02]
<Yunohost Git/Infra notifications> App mautrix_whatsapp failed all tests in job [#25775](https://ci-apps.yunohost.org/ci/job/25775) :(
[04:50:21]
<Yunohost Git/Infra notifications> [apps] tituspijean closed [pull request #2221](https://github.com/YunoHost/apps/pull/2221): Add Speckle Server to wishlist
[04:50:29]
<Yunohost Git/Infra notifications> [apps] tituspijean deleted branch add-to-wishlist-speckle-server
[09:48:04]
<tituspijean> oooh, I enabled auto-merging, so it's my name as committer, not the bot. My bad.
[09:48:04]
<tituspijean> mmh, the bot is in the bypass list though...
[09:48:04]
<tituspijean> Salamandar: adding dates to apps.toml now fails due to repository rules: https://github.com/YunoHost/apps/actions/runs/8756337511/job/24032570472 😕
[09:48:05]
<tituspijean> sooooo, we should *not* enable auto-merge
[09:48:05]
<Yunohost Git/Infra notifications> [apps] yunohost-bot pushed 1 commit to add-to-wishlist-quakejs: Add QuakeJS to wishlist ([d201c104](https://github.com/YunoHost/apps/commit/d201c104abb18e9de5655efac5de56652ded9174))
[09:48:06]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2250](https://github.com/YunoHost/apps/pull/2250): Add QuakeJS to wishlist
[09:48:06]
<Yunohost Git/Infra notifications> [apps] yunohost-bot labeled Wishlist on [pull request #2250](https://github.com/YunoHost/apps/pull/2250): Add QuakeJS to wishlist
[09:50:21]
<Yunohost Git/Infra notifications> [apps] tituspijean edited [pull request #2250](https://github.com/YunoHost/apps/pull/2250): Add QuakeJS to wishlist
[09:50:26]
<Yunohost Git/Infra notifications> [apps] tituspijean edited [pull request #2250](https://github.com/YunoHost/apps/pull/2250): Add QuakeJS to wishlist
[09:52:18]
<Yunohost Git/Infra notifications> [apps] tituspijean [commented](https://github.com/YunoHost/apps/pull/2250#issuecomment-2067622349) on [issue #2250](https://github.com/YunoHost/apps/pull/2250) Add QuakeJS to wishlist: No LICENSE file, but MIT mentioned in the README. Project seems abandoned with a last commit 5 years ago, which is not b...
[09:52:36]
<Yunohost Git/Infra notifications> [apps] tituspijean [commented](https://github.com/YunoHost/apps/pull/2250#issuecomment-2067622349) on [issue #2250](https://github.com/YunoHost/apps/pull/2250) Add QuakeJS to wishlist: No LICENSE file, but MIT mentioned in the README. Project seems abandoned with a last commit 5 years ago, which is not b...
[12:42:36]
<ChriChri> > <@chrichri:librem.one> I'd put this into an issue if nobody says it's waisted time and wouldn't be accepted as a PR anyway... ?!
https://github.com/YunoHost/issues/issues/2377
[13:18:32]
<rodinux> Hello, someone to approve this PR ? https://github.com/YunoHost-Apps/paheko_ynh/pull/58
[13:39:21]
<rodinux> Thanks a lot tituspijean
[14:44:35]
<rodinux> well I have a new problem with my development on paheko... We have
[15:18:48]
<Salamandar> > <@titus:pijean.ovh> Salamandar: adding dates to apps.toml now fails due to repository rules: https://github.com/YunoHost/apps/actions/runs/8756337511/job/24032570472 😕
Hmmmm did not think about that
[18:20:53]
<rodinux> Not really a problem, I found how resolve it, I am now adding a constant in the `config.local.php` which permit use a completion for addresses...
[18:27:39]
<rodinux> someone could valid my PR https://github.com/YunoHost-Apps/paheko_ynh/pull/59 ?
[18:34:52]
<Yunohost Git/Infra notifications> App lychee goes down from level 8 to 6 in job [#25792](https://ci-apps.yunohost.org/ci/job/25792)
[18:35:24]
<orhtej2> was there a recent UI change on Github? They now show WTF 7 letters of commit hash instead of 8
[18:35:30]
<orhtej2> or was it always this way?
[20:07:24]
<orhtej2> TFW you accidentally release an update to app because you forgot to check the base branch of a PR x_x
[20:14:39]
<tituspijean> > <@rodinux:matrix.org> someone could valid my PR https://github.com/YunoHost-Apps/paheko_ynh/pull/59 ?
So hum... cooperation is nice, but if you are the sole maintainer of an app and if you are pretty sure of what you are doing, feel free to bypass the branch protection rules. I see you are Admin of the repo, you may do that 😉
[20:16:10]
<rodinux> > <@titus:pijean.ovh> So hum... cooperation is nice, but if you are the sole maintainer of an app and if you are pretty sure of what you are doing, feel free to bypass the branch protection rules. I see you are Admin of the repo, you may do that 😉
Ok, I take the responsability since I could... thanks
[20:17:47]
<rodinux> I am surprise the branch testing is not protected, when merging I have the option to safely delete it...
[20:18:49]
<tituspijean> That's something I have been wondering about, I dunno if we can set both master and testing as not-deletable, and set master to allow PRs only and reject pushes
[20:19:04]
<tituspijean> for all repos, I mean
[20:20:09]
<rodinux> Ok, I understand, it could makes errors
[20:31:24]
<orhtej2> > <@titus:pijean.ovh> That's something I have been wondering about, I dunno if we can set both master and testing as not-deletable, and set master to allow PRs only and reject pushes
to disallow deleting mark both `testing` and `master` protected, to disallow pushes make `master` require PR as well
[20:32:05]
<orhtej2> (how to do it in all 500+ repos - beats me)
[20:32:32]
<orhtej2> perhaps some automation from API?
[20:32:47]
<tituspijean> pygithub would be able to do that
[20:32:59]
<orhtej2> doable: https://docs.github.com/en/rest/branches/branch-protection?apiVersion=2022-11-28#update-branch-protection
[20:33:10]
<orhtej2> yeah, probably bound in Python library as well
[20:33:46]
<orhtej2> (if you mark `testing` to require PR as well you're stuck in limbo if `testing` has conflicts with `master` branch, ask me how I know)
[20:34:38]
<tituspijean> I'd rather not 😆
[20:34:50]
<tituspijean> it's giving me *git shills*
[21:15:56]
<Josué> Hello, thatoo: are you here for https://github.com/YunoHost-Apps/synapse_ynh/issues/453
[21:28:38]
<thatoo> yes, I'm here
[22:41:07]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to update_app_levels: Update apps.toml: revert levels for grafana, halcyon, mantis, scovie ([62c48b61](https://github.com/YunoHost/apps/commit/62c48b6155444de7e5fe92349ea2abc85be859a2))
[22:41:34]
<Yunohost Git/Infra notifications> [apps] alexAubin edited [pull request #2249](https://github.com/YunoHost/apps/pull/2249): Update app levels according to CI results
[22:41:42]
<Yunohost Git/Infra notifications> [apps] alexAubin approved [pull request #2249](https://github.com/YunoHost/apps/pull/2249#pullrequestreview-2013186194) Update app levels according to CI results