Skip to content

Support PHP 8.5#38

Merged
lippserd merged 3 commits intomainfrom
support-php-85
Dec 16, 2025
Merged

Support PHP 8.5#38
lippserd merged 3 commits intomainfrom
support-php-85

Conversation

@sukhwinder33445
Copy link
Contributor

@sukhwinder33445 sukhwinder33445 commented Dec 1, 2025

PHP 8.5 related changes:

Other changes:

resolves #37

@cla-bot cla-bot bot added the cla/signed label Dec 1, 2025
@sukhwinder33445 sukhwinder33445 marked this pull request as draft December 1, 2025 14:52
@sukhwinder33445 sukhwinder33445 marked this pull request as ready for review December 4, 2025 12:44
@sukhwinder33445 sukhwinder33445 self-assigned this Dec 4, 2025
Copy link

@BastianLedererIcinga BastianLedererIcinga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@BastianLedererIcinga
Copy link

There is still one call to SplObjectStorage::contains in ValidatorChainTest.php, it would be nice to replace it, even if it's only a test.

@sukhwinder33445 sukhwinder33445 force-pushed the support-php-85 branch 2 times, most recently from 2ec22a0 to 1bece9a Compare December 12, 2025 07:40
@lippserd lippserd changed the title Add php 8.4/8.5 support Support PHP 8.5 Dec 15, 2025
sukhwinder33445 and others added 3 commits December 16, 2025 12:26
Otherwise, we would have to increase `guzzlehttp/psr7` (ddd7fa8) to support
recent PHP versions. This is a bit excessive for a library from which we only
need a single class exclusively for testing: The `UploadedFile` class was only
used to test our `FileValidator` and is now replaced by a minimal
implementation of `PSR-7#UploadedFileInterface`.
@lippserd lippserd merged commit 11ef35b into main Dec 16, 2025
13 checks passed
@lippserd lippserd deleted the support-php-85 branch December 16, 2025 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PHP 8.4/8.5 Support

3 participants