[01:07:56]
<Yunohost Git/Infra notifications> [package_check] @yalh76 commented on pull request #125 Make all tests: Solved it, now the result for https://github.com/YunoHost-Apps/nomad_ynh is: [image](https://user-images.githubusercon... https://github.com/YunoHost/package_check/pull/125#discussion_r969050506
[02:44:49]
<Yunohost Git/Infra notifications> [CI_package_check] @yalh76 created new branch arm64
[02:44:49]
<Yunohost Git/Infra notifications> [CI_package_check] @yalh76 pushed 1 commit to arm64: Fix arm64 architecture As there is no aarch54 at https://us.lxd.images.canonical.com/ https://github.com/YunoHost/CI_package_check/commit/30058abefd29702e13baaa573af1454c61756bae
[02:44:56]
<Yunohost Git/Infra notifications> [CI_package_check] @yalh76 opened pull request #42: Fix arm64 architecture https://github.com/YunoHost/CI_package_check/pull/42
[02:45:22]
<Yunohost Git/Infra notifications> [CI_package_check] @yalh76 pushed 2 commits to master: https://github.com/YunoHost/CI_package_check/compare/b32dbfcc0d5f...ea7546fbf9bd
[02:45:23]
<Yunohost Git/Infra notifications> [CI_package_check] @yalh76 merged pull request #42: Fix arm64 architecture https://github.com/YunoHost/CI_package_check/pull/42
[02:45:28]
<Yunohost Git/Infra notifications> [CI_package_check/master] Fix arm64 architecture As there is no aarch54 at https://us.lxd.images.canonical.com/ - yalh76
[02:45:28]
<Yunohost Git/Infra notifications> [CI_package_check] @yalh76 deleted branch arm64
[02:45:29]
<Yunohost Git/Infra notifications> [CI_package_check/master] Merge pull request #42 from YunoHost/arm64 Fix arm64 architecture - yalh76
[08:22:14]
<centralscrutinizer> yalh76: i have posted an issue on the mautrix whatsapp github page: https://github.com/YunoHost-Apps/mautrix_whatsapp_ynh/issues/70#issue-1369045688
It looks like you're the last actively taking care of the code, so I thought to give you a poke 🙂.
thanks in advance for your contribution
[08:35:32]
<yalh76> > yalh76: i have posted an issue on the mautrix whatsapp github page: https://github.com/YunoHost-Apps/mautrix_whatsapp_ynh/issues/70#issue-1369045688
> It looks like you're the last actively taking care of the code, so I thought to give you a poke 🙂.
> thanks in advance for your contribution
Sorry, this room is not the good place to discuss app issues, it should be in the support room.
I don’t actively taking care of Mautrix-WhatsApp. Only fixing a test issue since matrix_ynh changed one of the install question
[08:38:33]
<centralscrutinizer> > <@yalh:matrix.yalh.net> Sorry, this room is not the good place to discuss app issues, it should be in the support room.
> I don’t actively taking care of Mautrix-WhatsApp. Only fixing a test issue since matrix_ynh changed one of the install question
I see, thanks for the reply, and sorry for the intrusion 😶🌫️
[20:07:06]
<Tag> Mmmh that weird, code-server doesn't show up anymore in the catalog alors que https://github.com/YunoHost/apps/blob/master/apps.json#L432-L440 /o\
[20:38:46]
<Aleks> 🤔
[20:49:00]
<Aleks> `[Applist builder error] Processing code-server failed: unsupported operand type(s) for +: 'dict' and 'list'` 🤔
[20:53:21]
<Aleks> ah
[20:53:31]
<Aleks> ```
File "list_builder.py", line 329, in build_app_dict
set(manifest.get("antifeatures", []) + infos.get("antifeatures", []))
TypeError: unsupported operand type(s) for +: 'dict' and 'list'
```
[20:53:36]
<Aleks> https://github.com/YunoHost-Apps/code-server_ynh/blob/master/manifest.json#L19
[20:54:09]
<tituspijean> Wow that's a dict pic
[20:54:17]
<Aleks> xD
[20:54:25]
*tituspijean hides in the bush
[20:54:38]
<Aleks> 🥁📀
[20:55:50]
<Aleks> Tag: cf the stacktrace / manifest.json, I don't know what we actually want to do there ?
[20:56:16]
<Aleks> like do we want the list of keys from the dict ?
[20:56:44]
<Tag> ah
[20:56:56]
<Tag> 🤔
[20:57:23]
<Tag> let me dive in this code
[20:59:55]
<Tag> yeah only the lists of keys
[21:00:52]
<tituspijean> With the option to override their description, right?
[21:01:20]
<Tag> The description should not be included in the apps.json
[21:01:53]
<Tag> I mean, that's should be the actual behavior
[21:03:03]
<Tag> We override the description when we generate the README
[21:04:04]
<Tag> Aleks: something like `keys(manifest.get("antifeatures", {}))` ?
[21:04:50]
<Aleks> yup
[21:04:51]
<Aleks> ooooor
[21:05:03]
<Aleks> `list(manifest.get("antifeatures", {}).keys())` ?
[21:05:50]
<Tag> the result of `.keys()` isn't a list already ? 🤔 🧐
[21:06:15]
<Tag> ah, can be null ?
[21:06:45]
<Aleks> it's a generator
[21:06:59]
<Tag> Oh, python stuff ^^
[21:06:59]
<Tag> xD
[21:07:07]
<Aleks> (in python3* ;P)
[21:09:08]
<Yunohost Git/Infra notifications> [apps] @Tagadda pushed 1 commit to Tagadda-patch-1: Fix a bug introduced with antifeatures https://github.com/YunoHost/apps/commit/5278251b1d580dad25ba08f20d272530e4fba1ed
[21:09:08]
<Yunohost Git/Infra notifications> [apps] @Tagadda created new branch Tagadda-patch-1
[21:13:20]
<Yunohost Git/Infra notifications> [apps] @Tagadda opened pull request #1515: Fix a bug introduced with antifeatures https://github.com/YunoHost/apps/pull/1515