tests.yml: Add PHP 8.5

[reedy]

No description provided.

[alecpl]

Thanks. Looks like all tests fail because of PHP Deprecated: PHP Startup: Directive 'report_memleaks' is deprecated in Unknown on line 0.

[alecpl]

Try this:

--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -25,6 +25,7 @@ jobs:
         with:
           php-version: ${{ matrix.php }}
           coverage: none
+          ini-values: display_startup_errors=off
 
       - name: Execute tests
         run: pear run-tests tests/*.phpt

[alecpl]

I see you're already on it pear/pear-core#161

[ashnazg]

I see you're already on it pear/pear-core#161

@alecpl , fixed in pear-core v1.10.18.

[alecpl]

Ouch. I merged it, but we need a new release of pear/pear-core-minimal to work.

[ashnazg]

Ouch. I merged it, but we need a new release of pear/pear-core-minimal to work.

Didn't realize anyone was explicitly tying minimal release #s to core release #s for workflows... I've updated my release steps to always do an accompanying minimal tag going forward... I've now created minimal tags for .17 and .18.

[reedy]

Guess it would be worth re-running tests to confirm they're passing and there's no other issues wrt PHP 8.5

[alecpl]

Damn. It seems we're not using composer in CI. I tried my display_startup_errors=off change from the comment above, it does not seem to help. So, I guess we need either make tests use composer dependencies or update Pear in the system somehow.

[alecpl]

Solved. Tests pass in master.