YunoHost-Apps / synapse_ynh

Matrix server (synapse) package for YunoHost
https://matrix.org/
GNU General Public License v3.0
79 stars 42 forks source link

Prep app for YunoHost 11.1 #348

Closed tituspijean closed 1 year ago

tituspijean commented 1 year ago

PR Status

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

Josue-T commented 1 year ago

Are there a solution to keep the compatibility between yunohost <11.1 and yunohost >=11.1 ?

tituspijean commented 1 year ago

We could do a simple if 11.1 then..., or patch the core to automagically fill -F if both -l and -n are provided. What do you think @alexAubin ?

alexAubin commented 1 year ago

FYI I added a patch for automagic backward compatibility : https://github.com/YunoHost/yunohost/commit/d254fb1b058340b1905fa0438704b45b852827f3

tituspijean commented 1 year ago

I can close this then, I guess? :)