search

krakjoe / uopz

User Operations for Zend
Other
358 stars 47 forks source link

CI: test on PHP 8.1, 8.2, 8.3 #171

Closed SMillerDev closed 3 months ago

SMillerDev commented 1 year ago

Also, is there any reason not to use https://github.com/setup-php/setup-php ?

cmb69 commented 3 months ago

Closing and re-opening to retrigger CI.

SMillerDev commented 3 months ago

Updated to the latest PHP versions

cmb69 commented 3 months ago

Also, is there any reason not to use https://github.com/setup-php/setup-php ?

Not exactly sure, but I mean to remember that this caused some issues in the past.

Updated to the latest PHP versions

Wow, thanks, that was fast. However, there is apparently a cron schedule in place for running CI; not sure if that's reasonable. Anyway, I'll try to keep an eye on this.

cmb69 commented 3 months ago

Okay, CI is running now; let's see how this goes.

cmb69 commented 3 months ago

Due to fail fast behavior, I'm not 100% sure, but at least on Windows tests/007.phpt fails with PHP 8.3:

--
         int(5)
       }
       ["bar"]=>
016-   string(3) "bar"
016+   NULL
     }
     array(2) {
       ["vars"]=>
--

I suggest to test with PHP 8.0, 8.1 and 8.2 for now.

cmb69 commented 3 months ago

Thank you again! I have applied this via https://github.com/krakjoe/uopz/commit/7b70bbf0c23b772dd3fbeb52f461749c166e6b29.