YunoHost-Apps / mautrix_telegram_ynh

Matrix Telegram pupetting bridge for YunoHost
https://github.com/mautrix/telegram
GNU General Public License v3.0
14 stars 4 forks source link

try to fix CI #6

Closed benneti closed 2 years ago

benneti commented 2 years ago

Problem

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

benneti commented 2 years ago

@ericgaspar I do not know why you closed the previous one, so here a new one... (If me rewriting the history caused problems I am sorry for that, but as this might gain some tracktion I prefer it if my commits are from the email address I use with github) Maybe @Gredin67 could help with the CI problems

EDIT: also is there a VM or similar such that I can try the CI on my desktop (which runs on nixos so no apt available), but I guess the simplest way is to set up a debian or ubuntu vm to run the tests

benneti commented 2 years ago

I think I finally figured out the problem, mautrix_telegram fails to generate the synapse config when I use the values from the example configuration...

benneti commented 2 years ago

locally all tests (apart from port already in use pass)

benneti commented 2 years ago

!testme

benneti commented 2 years ago

Screenshot from 2021-09-10 16-57-29

ericgaspar commented 2 years ago

!testme

yunohost-bot commented 2 years ago

:sunflower: Test Badge

benneti commented 2 years ago

thanks! Tests show all green https://ci-apps-dev.yunohost.org/ci/job/3109 merging this into master as it improves the quality a bit.

There is some stuff left like rewriting the readme via the manifest automation but I do not have time for that right now