YunoHost-Apps / mautrix_signal_ynh

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

Upgrade to v0.2.1 #32

Closed nathanael-h closed 2 years ago

nathanael-h 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

nathanael-h commented 2 years ago

Hello @Gredin67 or @MayeulC could you trigger the CI and review ?

MayeulC commented 2 years ago

Hello nathanael! Thanks for the PR!

I will review, but I can't trigger the CI. I asked for the rights in the appropriate channel some time ago, but didn't get feedback.

Please note that this will still not upgrade signald #22 , probably due to a bug in yunohost's helper scripts. If you run signald < 0.15.0-27 I suggest you manually install it from the release deb at https://updates.signald.org/pool/main/s/signald/signald_0.15.0-27-88722d97_amd64.deb :)

nathanael-h commented 2 years ago

Ok thank for details about signald, and the review. Could we merge this and deal with signald upgrade in another PR ? Or is this mandatory ?

nathanael-h commented 2 years ago

In fact I am running this version: signald 0.15.0-27-88722d97 Maybe I have it since I used the PR https://github.com/YunoHost-Apps/mautrix_signal_ynh/pull/28#issuecomment-979870861

nathanael-h commented 2 years ago

!testme

yunohost-bot commented 2 years ago

Meow :cat2: Test Badge

MayeulC commented 2 years ago

Could we merge this and deal with signald upgrade in another PR

That's the plan.

I'm not sure what the merging policy is. I will merge into testing, then merge to master in a few days.

Thanks for triggering the CI, and giving me the rights :)