- Issue created by @Grimreaper
- @grimreaper opened merge request.
- 🇫🇷France pdureau Paris
Florent,
is your proposal compatible with the interface public static function normalize(mixed $value): mixed; ?
I have read https://php.watch/versions/8.0/mixed-type but I am not sure I understand everything
- 🇫🇷France Grimreaper France 🇫🇷
I have tested without changing the interface and it was ok.
https://php.watch/versions/8.0/mixed-type#mixed-variance
https://php.watch/versions/8.0/mixed-type#mixed-covarianceB::baz: Allowed: Narrows down the declared mixed type.
I don't know the current code coverage of the tests but I guess a tests would have failed if this change broke something.
- 🇫🇷France pdureau Paris
I don't know the current code coverage of the tests but I guess a tests would have failed if this change broke something.
This method is not currently covered by a test, as far as I know
-
grimreaper →
committed 2dab18e8 on 2.0.x
Issue #3485900 by grimreaper: Fix normalize typing
-
grimreaper →
committed 2dab18e8 on 2.0.x
Automatically closed - issue fixed for 2 weeks with no activity.