Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update require php to ~8.4.0 #118

Closed
wants to merge 1 commit into from

Conversation

raphaelstolt
Copy link

@raphaelstolt raphaelstolt commented Jul 30, 2024

Makes the laminas-stdlib available for PHP 8.4.

Signed-off-by: Raphael Stolt <[email protected]>
@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
Status: Done
Development

Successfully merging this pull request may close these issues.

6 participants