YunoHost-Apps / mautrix_signal_ynh

Matrix signal package for YunoHost
GNU Affero General Public License v3.0
4 stars 10 forks source link

Upgrade mautrix-signal to v0.3.0 #47

Closed MayeulC closed 2 years ago

MayeulC commented 2 years ago

Problem

Solution

PR Status

Package_check results


If you have access to App Continuous Integration for packagers you can provide a link to the package_check results like below, replacing '-NUM-' in this link by the PR number and USERNAME by your username on the ci-apps-dev. Or you provide a screenshot or a pastebin of the results

Build Status

MayeulC commented 2 years ago

!testme

yunohost-bot commented 2 years ago

:sunflower: Test Badge

MayeulC commented 2 years ago

Hmm, besides Synapse falling to install in the CI, an issue might be that upstream is removing support for python 3.7 https://github.com/mautrix/signal/commit/c92d4f1818462b66a73ab57ea1bc063e4514be30 (yes, it's a typo in the commit title). This was bumped due to mautrix requiring python > 3.7: https://github.com/mautrix/python/releases/tag/v0.15.0

Gredin67 commented 2 years ago

!bullseyetestme

yunohost-bot commented 2 years ago

[Bullseye :horse:] :rocket: Test Badge

MayeulC commented 2 years ago

It might need rebasing on #45 before...

Edit: ah, you changed the target branch. Got it!

Gredin67 commented 2 years ago

@MayeulC how about taking over the Maintainer status in manifest.json ? As yunohost bullseye will soon be released, we could push bullseye to the testing branch. I just created a buster branch from the testing branch

MayeulC commented 2 years ago

I think we should add some checks to fail the upgrade on buster before merging this into testing or master.

I haven't upgraded my yunohost to bullseye yet, I'm not sure about the maintainer status. I imagine I can, but I would prefer to co-maintain it, especially as I won't have much time until the end of July. Going further, I am not sure if I will keep using Yunohost for my Matrix infrastructure (I am not sure of anything yet).