laminas-stdlib icon indicating copy to clipboard operation
laminas-stdlib copied to clipboard
verified publisher icon laminas

Update require php to ~8.4.0

Open raphaelstolt opened this issue 1 year ago • 8 comments

Makes the laminas-stdlib available for PHP 8.4.

raphaelstolt avatar Jul 30 '24 13:07 raphaelstolt

8.4 is not yet available in CI. The supported range won't be updated until we can verify that all tests pass on 8.4

gsteel avatar Jul 30 '24 13:07 gsteel

It's available.

raphaelstolt avatar Jul 30 '24 13:07 raphaelstolt

It's available.

Show me in the checks below where this lib was tested on 8.4

gsteel avatar Jul 30 '24 14:07 gsteel

I can just link a GitHub Actions job where PHP 8.4 is successfully installed, so generally it's available.

raphaelstolt avatar Jul 30 '24 14:07 raphaelstolt

Needs to be in:

  • https://github.com/laminas/laminas-ci-matrix-action
  • https://github.com/laminas/laminas-continuous-integration-action

Ocramius avatar Jul 30 '24 16:07 Ocramius

@raphaelstolt the issue seem to be adressed, can you run the ci again

fezfez avatar Aug 03 '24 17:08 fezfez

Needs adjusting 😁

Probably need to ignore platform requirements on PHP 8.4

Ocramius avatar Aug 04 '24 10:08 Ocramius

Are we waiting for vemio/psalm PR with PHP 8.4 support to get merged for this to unblock ?

nishant04412 avatar Oct 01 '24 05:10 nishant04412

Thanks for your help here @raphaelstolt but this has been implemented in #119

gsteel avatar Oct 29 '24 13:10 gsteel