YunoHost-Apps / synapse_ynh

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

Testing #444

Closed Josue-T closed 2 months ago

Josue-T commented 3 months ago

Testing

Too many change to describe here. For more info see the previous merged PR

PLEASE don't merge this unitls these PR are merged and released:

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)

alexAubin commented 3 months ago

(I took the liberty to iterate on ADMIN.md, initially because the top 'install' section should be a PRE_INSTALL.md, and ended up reworking some of the sections and wording)

alexAubin commented 2 months ago

!testme

yunohost-bot commented 2 months ago

Alrighty! Test Badge

yunohost-bot commented 2 months ago

:bug: Test Badge

alexAubin commented 2 months ago

!testme

yunohost-bot commented 2 months ago

:books: Test Badge

yunohost-bot commented 2 months ago

Meow :cat2: Test Badge

Josue-T commented 2 months ago

Merged the linter fix, LGTM to merge, ideally with this small comment to improve minor rendering

This fixes bookworm, which in turn will allow to work on all the mautrix bridges apps which for now are broken on bookworm because synapse can't be installed

Thanks for the review and the feedback. To the only blocking issue to merge this it that https://github.com/YunoHost/yunohost/pull/1789 still not released on core side and so for now the config panel is just completely broken without this.

alexAubin commented 2 months ago

the only blocking issue to merge this it that https://github.com/YunoHost/yunohost/pull/1789 still not released on core side and so for now the config panel is just completely broken without this.

Releasing in 11.2.11, will also update the CI images :+1:

alexAubin commented 2 months ago

(Done)