Skip to content

Update require php to ~8.4.0#118

Closed
raphaelstolt wants to merge 1 commit intolaminas:3.20.xfrom
raphaelstolt:require-php-8.4
Closed

Update require php to ~8.4.0#118
raphaelstolt wants to merge 1 commit intolaminas:3.20.xfrom
raphaelstolt:require-php-8.4

Conversation

@raphaelstolt
Copy link

@raphaelstolt raphaelstolt commented Jul 30, 2024

Makes the laminas-stdlib available for PHP 8.4.

Signed-off-by: Raphael Stolt <r.stolt@esvmedien.de>
@gsteel
Copy link
Member

gsteel commented Jul 30, 2024

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

@raphaelstolt
Copy link
Author

It's available.

@gsteel
Copy link
Member

gsteel commented Jul 30, 2024

It's available.

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

@raphaelstolt
Copy link
Author

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

@Ocramius
Copy link
Member

@fezfez
Copy link

fezfez commented Aug 3, 2024

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

@Ocramius
Copy link
Member

Ocramius commented Aug 4, 2024

Needs adjusting 😁

Probably need to ignore platform requirements on PHP 8.4

@nishant04412
Copy link

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

@gsteel
Copy link
Member

gsteel commented Oct 29, 2024

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

@gsteel gsteel closed this Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants