YunoHost-Apps / rss-bridge_ynh

RSS-Bridge package for YunoHost
https://github.com/RSS-Bridge/rss-bridge
The Unlicense
14 stars 5 forks source link

Php8.1 #57

Closed lapineige closed 1 year ago

lapineige commented 1 year ago

Replacing #50, thank you @Binnette for the initial work :)

Problem

PHP 7 have reached end of life: https://www.php.net/supported-versions.php

Solution

Package upgraded to use PHP 8.

PHP 8 is supported by RSS-Bridge since version 2022-01-20. And this YNH package already have a newer version.

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)

lapineige commented 1 year ago

!testme

yunohost-bot commented 1 year ago

:sunflower: Test Badge

lapineige commented 1 year ago

!testme

yunohost-bot commented 1 year ago

:stuck_out_tongue_winking_eye: Test Badge

lapineige commented 1 year ago

!testme

yunohost-bot commented 1 year ago

:stuck_out_tongue_winking_eye: Test Badge

lapineige commented 1 year ago

!testme

yunohost-bot commented 1 year ago

:v: Test Badge

lapineige commented 1 year ago

I'm going to add them one by one apparently ^^

I wonder why this is needed in the CI with php8(.1?) and not with php 7.3 as previously used. Anyway, it's important that we list our dependencies no matter what the system already has.