Change minimum requirement to PHP 8 and add type hints

spatie / fractalistic

A framework agnostic, developer friendly wrapper around Fractal

https://freek.dev/644-a-developer-friendly-wrapper-around-fractal

MIT License

382 stars 32 forks source link

Change minimum requirement to PHP 8 and add type hints #68

Closed shahlin closed 1 year ago

shahlin commented 1 year ago

This PR contains:

[x] PHP 8 required as minimum PHP version
[x] Updated code to have type hints for properties, method arguments, return types.
[x] Initialized properties with null value where necessary
[x] Updated SparseFieldsetsTest to correctly follow the method signature of Fractal::item() in test (identified after adding type hints)
[ ] Remove docblocks

spatie-bot commented 1 year ago

Dear contributor,

because this pull request seems to be inactive for quite some time now, I've automatically closed it. If you feel this pull request deserves some attention from my human colleagues feel free to reopen it.