Add support for Symfony 8, require at least PHP 8.1 and add PHP 8.5 to CI

[alexander-schranz]

Test current compatibility to Symfony 8.

Is required: FriendsOfSymfony/FOSHttpCache#597 as CI installs this package.

[alexander-schranz]

@Toflar CI seems currently be disabled. Github is doing this because there where not any changes and disable CI with schedules after 1-2 month:

psr6-symfony-http-cache-store/.github/workflows/ci.yaml

Lines 10 to 11 in 9b3eb5b

	schedule:
	- cron: 0 13 * * MON

You should be able reactivate it in: https://github.com/Toflar/psr6-symfony-http-cache-store/actions

[Toflar]

Re-enabled, I guess you need to push again to trigger CI?

[alexander-schranz]

@Toflar how would you handle the deprecation errors on lowest? As I added PHP 8.4:

Lot of Implicitly marking parameter like are shown:

Implicitly marking parameter $value as nullable is deprecated, the explicit nullable type must be used instead

[Toflar]

I guess that's a wrong configuration of phpunit because those are not deprecations we can influence, are they? So update the phpunit configuration, maybe?

[dbu]

for those test failures: my solution was to require phpunit ^9.6.29 as that avoided the deprecations that are treated as failure here. can you try that?

[dbu]

@Toflar can you please allow this to build again?

[alexander-schranz]

Looks like CI is blocked again. CI Need be reenabled here: https://github.com/Toflar/psr6-symfony-http-cache-store/actions

This is because of

psr6-symfony-http-cache-store/.github/workflows/ci.yaml

Lines 10 to 11 in 9b3eb5b

	schedule:
	- cron: 0 13 * * MON

and Github after a while just disable the whole actions if schedule is enabled on a repo without any activity.

[Toflar]

Done!

[alexander-schranz]

@Toflar by require the new phpunit bridge versions we would require to remove PHP 8.0 support? Should we investigate to still support PHP 8.0 or remove support of it?

Sulus currently PHP minimum version is 8.2.

[dbu]

8.0 is EOL. i suggest we bump to 8.1
also, latest FOSHttpCache requires 8.1

[dbu]

i don't see anything setting the composer-stability. i don't think we need this

[alexander-schranz]

I see, think make still sense to test against pre versions. But doing now only in a 8.6 task.

[Toflar]

We can bump the dependencies for sure, no need to force things for old stuff.

[dbu]

looks good to me.

[Toflar]

Thanks for the work guys! Is it enough to release this as 4.3.0 or would you actually prefer having a major (which could allow for more cleanup if needed?).

[alexander-schranz]

I'm fine with releasing it as 4.3.0.

[Toflar]

Done, thank you! https://github.com/Toflar/psr6-symfony-http-cache-store/releases/tag/4.3.0