Saturday, January 07, 2023
apps@conference.yunohost.org
January
Mon Tue Wed Thu Fri Sat Sun
            1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
         

[07:39:58] <tituspijean> https://forum.yunohost.org/t/most-applications-gone-after-php-upgrade/22969

Could it be because the address of some repos was "fixed" recently (mostly capitalization)?

I'll investigate but I'm on mobile so a bit slow
[07:44:29] <tituspijean> Mmmh does not look like it's the reason.
[08:26:39] <tituspijean> There seems to be a discrepancy between YunoHost/apps/apps.json and the one generated at https://app.yunohost.org/default/v3/apps.json
[08:28:55] <tituspijean> app.yunohost.org:
```
monica
...
maintained true
...
zerotier
maintained false
```

YunoHost/apps/apps.json:
```
"monica": {
"category": "wat",
"level": 7,
"state": "working",
"url": "https://github.com/YunoHost-Apps/monica_ynh",
"antifeatures": [
"package-not-maintained"
]
},
...
"zerotier": {
"antifeatures": [
"not-totally-free",
"non-free-network"
],
"category": "system_tools",
"level": 6,
"state": "working",
"subtags": [
"network"
],
"url": "https://github.com/YunoHost-Apps/zerotier_ynh"
},
```
[08:41:32] <tituspijean> Maybe a bug in the list builder?
[08:44:00] <tituspijean> haaaaaa https://github.com/YunoHost/apps/blob/5033ee072282b56eef21bc8d813c34573555c98a/list_builder.py#L336
[08:44:36] <tituspijean> ```
"maintained": 'package-not-maintained' in infos.get('antifeatures'),
```

returns `maintained: true` if "not maintained" is an antifeature 🙃
[08:46:00] <Yunohost Git/Infra notifications> [apps] @tituspijean created new branch fix-catalog
[08:46:01] <Yunohost Git/Infra notifications> [apps] @tituspijean pushed 1 commit to fix-catalog: Fix maintainership flag https://github.com/YunoHost/apps/commit/9cff78e69581eee10da0f71ec13ac6ed88589503
[08:46:11] <Yunohost Git/Infra notifications> [apps] @tituspijean opened pull request #1585: Fix maintainership flag https://github.com/YunoHost/apps/pull/1585
[08:46:25] <tituspijean> Aleks (he/him/il/lui) , care to comment? 😛
[09:12:18] <Yunohost Git/Infra notifications> [apps] @tituspijean pushed 1 commit to fix-catalog: Fix maintainership flag² https://github.com/YunoHost/apps/commit/6cccd3d6befb733178f7810cbc5b2b1fd807511a
[09:13:42] <Yunohost Git/Infra notifications> [apps] @tituspijean commented on issue #1585 Fix maintainership flag: Fix yolo-implemented and tested, merging with admin rights. https://github.com/YunoHost/apps/pull/1585#issuecomment-1374422682
[09:13:54] <Yunohost Git/Infra notifications> [apps] @tituspijean merged pull request #1585: Fix maintainership flag https://github.com/YunoHost/apps/pull/1585
[09:13:54] <Yunohost Git/Infra notifications> [apps] @tituspijean pushed 1 commit to master: Fix maintainership flag (#1585) https://github.com/YunoHost/apps/commit/3080cb0d49b4b11a9fbe391886ce91af0ad1a7b1
[09:13:57] <Yunohost Git/Infra notifications> [apps] @tituspijean deleted branch fix-catalog
[12:22:21] <florent> Wonder why the elasticsearch package is referenced as broken 🤔
[12:22:24] <florent> https://aria.im/_matrix/media/v1/download/matrix.skeptikon.fr/eEYQBQMctTeVCUMlGfVcnkvX
[12:27:59] <tituspijean> I think it's because the Official CI has not assigned its level yet
[12:41:56] <@oufmilo:mtx.travelpandas.eu> Is there a problem whit the official ci ? Because nothing is running actually
[12:42:40] <@oufmilo:mtx.travelpandas.eu> Hello 👋
Is there a problem with the official CI ? Because nothing is running actually
[12:45:06] <Aleks (he/him/il/lui)> > <@titus:pijean.ovh> Aleks (he/him/il/lui) , care to comment? 😛

🙀
[12:45:49] <Aleks (he/him/il/lui)> cheers for merging
[12:45:57] <Aleks (he/him/il/lui)> not sure to understand why adding rich though o_O
[12:46:07] <tituspijean> oooops it was for debugging
[12:46:10] <tituspijean> I forgot to remove it
[12:46:33] <tituspijean> (but the script already handles debugging well)
[12:46:49] <Aleks (he/him/il/lui)> also `infos.get('antifeatures', False)` -> you probably meant `infos.get('antifeatures', [])` because `foo in False` is gonna crash
[12:47:36] <tituspijean> Mmmh it did not yell at me when I ran it this morning, but feel free to fix it!
[12:47:36] <Aleks (he/him/il/lui)> (and my code had this issue too because `foo in None` doesn't work either 😅)
[12:47:47] <tituspijean> (I'm in a train, connection is... sncf'esque)
[12:49:09] <tituspijean> 🅞🅤🅕🅜🅘🅛🅞 it is running, though job https://ci-apps.yunohost.org/ci/job/12607 is somehow not displayed in the index...
[12:49:21] <Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to master: Smol fixes in list_builder.py https://github.com/YunoHost/apps/commit/4bf45bad389fa5d98705b1dd16539aa09f74213b
[12:50:51] <tituspijean> I manually edited the script to try it out on Saperlipopette, would that prevent the auto git pull?
[12:51:22] <florent> > <@titus:pijean.ovh> I think it's because the Official CI has not assigned its level yet

I see. Thanks!
[12:53:05] <florent> Hope I don't bother. I packaged OpenSearch before ElasticSearch, the first is not referenced in this page: https://yunohost.org/fr/apps
I wonder why. If I just have to wait, no problem for me though :)
[12:53:52] <Aleks (he/him/il/lui)> i do see opensearch though
[12:55:06] <Yunohost Git/Infra notifications> [apps] @alexAubin edited pull request #1574: logos: remove curves https://github.com/YunoHost/apps/pull/1574
[12:55:17] <Yunohost Git/Infra notifications> [apps] @alexAubin pushed 2 commits to master: https://github.com/YunoHost/apps/compare/4bf45bad389f...77d4be492905
[12:55:17] <Yunohost Git/Infra notifications> [apps] @alexAubin merged pull request #1574: logos: remove curves https://github.com/YunoHost/apps/pull/1574
[12:55:20] <Yunohost Git/Infra notifications> [apps/master] remove curves - Éric Gaspar
[12:55:23] <Yunohost Git/Infra notifications> [apps/master] Merge pull request #1574 from YunoHost/remove-curves remove curves - Alexandre Aubin
[12:55:49] <Yunohost Git/Infra notifications> [apps] @ericgaspar deleted branch remove-curves
[13:07:45] <Yunohost Git/Infra notifications> [apps] @tituspijean commented pull request #1584 Update apps level: - [ ] Flarum should be level 8 (connection error in last test): https://ci-apps.yunohost.org/ci/job/11776 https://github.com/YunoHost/apps/pull/1584#pullrequestreview-1239669020
[13:13:24] <Yunohost Git/Infra notifications> [apps] @tituspijean edited review pull request #1584: Update apps level https://github.com/YunoHost/apps/pull/1584#pullrequestreview-1239669434
[13:13:24] <Yunohost Git/Infra notifications> [apps] @tituspijean changes_requested pull request #1584 Update apps level https://github.com/YunoHost/apps/pull/1584#pullrequestreview-1239669434
[13:13:24] <Yunohost Git/Infra notifications> [apps] @tituspijean commented on pull request #1584 Update apps level: suggestion "level": 8, as per https://ci-apps.yunohost.org/ci/job/12598 https://github.com/YunoHost/apps/pull/1584#discussion_r1064002776
[13:52:05] <@oufmilo:mtx.travelpandas.eu> Another question, do you know why caclkey package is referenced as broken ? The official CI is ok since yesterday and I don't know why the 2 badges are broken too (maintainance status and working status)
[13:54:43] <tituspijean> The CI may be OK, but its level has not been updated into apps.json yet.
[13:55:33] <tituspijean> cf. https://github.com/YunoHost/apps/pull/1584/files#diff-70e375511009c0cb767fdc64667e148359c6d6b7b8cf4f100b7eb0e853f83529R406-R416
[13:57:44] <tituspijean> The process is:

1. you do your thing in the repo of your app
2. if the master branch (or main branch as specified in apps.json) is HEAD commit is changed, then a job is triggered in the official CI (check is every 10 min IIRC), or once every month.
3. the level is calculated and stored
4. level changes are pushed to YunoHost/apps every Friday
5. we review it over the weekend and merge for official catalog update
[13:57:58] <tituspijean> The process is:

1. you do your thing in the repo of your app
2. if the master branch (or main branch as specified in apps.json) HEAD commit is changed, then a job is triggered in the official CI (check is every 10 min IIRC), or once every month.
3. the level is calculated and stored
4. level changes are pushed to YunoHost/apps every Friday
5. we review it over the weekend and merge for official catalog update
[14:00:26] <@oufmilo:mtx.travelpandas.eu> > <@titus:pijean.ovh> The process is:
>
> 1. you do your thing in the repo of your app
> 2. if the master branch (or main branch as specified in apps.json) HEAD commit is changed, then a job is triggered in the official CI (check is every 10 min IIRC), or once every month.
> 3. the level is calculated and stored
> 4. level changes are pushed to YunoHost/apps every Friday
> 5. we review it over the weekend and merge for official catalog update

Thank you for the clarification 🙏
[14:09:40] <florent> > <@Alekswag:matrix.org> i do see opensearch though

Hmm, indeed! Thanks, please forget my question ^^'
[16:41:05] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 3 commits to master: https://github.com/YunoHost/apps/compare/77d4be492905...4bc835f7ba33
[16:41:06] <Yunohost Git/Infra notifications> [apps/master] Create uptime-kuma.png - Éric Gaspar
[16:41:10] <Yunohost Git/Infra notifications> [apps/master] Create yacy.png - Éric Gaspar
[16:41:10] <Yunohost Git/Infra notifications> [apps/master] Create whitebophir.png - Éric Gaspar
[16:44:06] <Yunohost Git/Infra notifications> [apps] @tituspijean pushed 1 commit to modify_level: Flarum is level 8 https://github.com/YunoHost/apps/commit/6cd8d8029429acfc3675b0b94da1b161a9aec9ac
[16:44:21] <Yunohost Git/Infra notifications> [apps] @tituspijean edited review pull request #1584: Update apps level https://github.com/YunoHost/apps/pull/1584#pullrequestreview-1239669020
[16:44:54] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 2 commits to master: https://github.com/YunoHost/apps/compare/4bc835f7ba33...961b113bf900
[16:44:57] <Yunohost Git/Infra notifications> [apps/master] Create nocodb.png - Éric Gaspar
[16:45:02] <Yunohost Git/Infra notifications> [apps/master] Create openproject.png - Éric Gaspar
[16:46:13] <Yunohost Git/Infra notifications> [apps] @tituspijean pushed 1 commit to modify_level: Ghost is level 8 https://github.com/YunoHost/apps/commit/cafa2f658c60e01efdaf369dbd741599bca80e3c
[16:49:50] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to master: Create ums.png https://github.com/YunoHost/apps/commit/e1b7e6e03832f22b425812c7e5892f39df154069
[16:54:39] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to master: Create mailman3.png https://github.com/YunoHost/apps/commit/0df2c0d53ace4e38b58c98a613dfb8f0238c8328
[16:57:09] <eric_G[m]> crap... I realize I'm sending this to master 😅
[16:58:10] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 31 commits to add-missing-logos: https://github.com/YunoHost/apps/compare/b3de2e6d0a2b...fe2408ac2bc5
[16:58:14] <Yunohost Git/Infra notifications> [apps/add-missing-logos] Create ums.png - Éric Gaspar
[16:58:16] <Yunohost Git/Infra notifications> [apps/add-missing-logos] Create mailman3.png - Éric Gaspar
[16:58:22] <Yunohost Git/Infra notifications> [apps/add-missing-logos] Merge branch master into add-missing-logos - Éric Gaspar
[17:00:49] <Yunohost Git/Infra notifications> [apps] @ericgaspar merged pull request #1575: Add missing logos https://github.com/YunoHost/apps/pull/1575
[17:00:51] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 49 commits to master: https://github.com/YunoHost/apps/compare/0df2c0d53ace...6f5bacc5ab70
[17:00:53] <Yunohost Git/Infra notifications> [apps] @ericgaspar deleted branch add-missing-logos
[17:00:53] <Yunohost Git/Infra notifications> [apps/master] Create pico.png - Éric Gaspar
[17:00:59] <Yunohost Git/Infra notifications> [apps/master] Merge branch master into add-missing-logos - Éric Gaspar
[17:01:04] <Yunohost Git/Infra notifications> [apps/master] Merge pull request #1575 from YunoHost/add-missing-logos Add missing logos - Éric Gaspar
[17:15:37] <Yunohost Git/Infra notifications> [apps] @tituspijean edited review pull request #1584: Update apps level https://github.com/YunoHost/apps/pull/1584#pullrequestreview-1239688769
[17:15:38] <Yunohost Git/Infra notifications> [apps] @tituspijean changes_requested pull request #1584 Update apps level https://github.com/YunoHost/apps/pull/1584#pullrequestreview-1239688769
[17:15:39] <Yunohost Git/Infra notifications> [apps] @tituspijean commented on pull request #1584 Update apps level: suggestion "category": "dev", "level": 6, "state": "working", as per https://ci-apps.yun... https://github.com/YunoHost/apps/pull/1584#discussion_r1064026008
[17:15:39] <Yunohost Git/Infra notifications> [apps] @tituspijean commented on pull request #1584 Update apps level: suggestion "level": 6, as per https://ci-apps.yunohost.org/ci/job/12512 Upgrades tests will most likely ... https://github.com/YunoHost/apps/pull/1584#discussion_r1064026421
[17:33:44] <Yunohost Git/Infra notifications> [apps] @tituspijean pushed 1 commit to modify_level: ElasticSearch7 and Kimai2 are level 6 https://github.com/YunoHost/apps/commit/1dd88e88de2b08be5ce54d1bec9a55243b69c956
[17:33:56] <Yunohost Git/Infra notifications> [apps] @tituspijean approved pull request #1584 Update apps level https://github.com/YunoHost/apps/pull/1584#pullrequestreview-1239690624
[22:32:02] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to master: add logos https://github.com/YunoHost/apps/commit/06af7121d943abee85b4007ea9a2148ffc6f6ebb
[22:44:54] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to master: add logos https://github.com/YunoHost/apps/commit/bffc2e8b2e1d6f993a19d849f355caa91ef2d7ad