PHP 8.2 compatibility (and some more)

prooph / event-store

PHP 7.4 EventStore Implementation

http://getprooph.org

BSD 3-Clause "New" or "Revised" License

548 stars 73 forks source link

PHP 8.2 compatibility (and some more) #430

Closed zluiten closed 1 year ago

zluiten commented 1 year ago

Most of these changes are done with the help of rector.

Ensure compatibility with PHP 8.2 (dynamic properties)
Add explicit public visibility to remaining constants
Use array spread operator instead of array_merge where possible
Add null default to nullable properties properties that have no default
Add typed properties and return types where possible
Simplify empty() function calls on empty arrays
Update licenses to 2023

fritz-gerneth commented 1 year ago

Not having checked every single file, but looks good to me

prolic commented 1 year ago

Thanks @zluiten! Great work!

prolic commented 1 year ago

Currently some builds are failing, @zluiten if you have time to look into those, would be great. Otherwise I try to check when I have time. I'd like to have all green builds before a release.

zluiten commented 1 year ago

Thanks @prolic for the quick review/merge! The fix is here #431.