Does it work with PHP8.1 ?

[smknstd]

Does it work with PHP8.1 ?

[vesper8]

@grimzy It would be great if you tagged a small update taking care of these deprecations before jumping on to the Laravel 9 release, which you already have a PR for. This is because a lot of users will be using Laravel 8 with PHP 8.1 for a long time still before upgrading to Laravel 9.

[jangaraev]

Have the same issue after switching to PHP8.1.

Return type of Grimzy\LaravelMysqlSpatial\Types\Point::jsonSerialize() should either be compatible with 
JsonSerializable::jsonSerialize(): mixed, or the #[\ReturnTypeWillChange] attribute should be used to
temporarily suppress the notice in vendor/grimzy/laravel-mysql-spatial/src/Types/Point.php on line 95

[smknstd]

https://github.com/grimzy/laravel-mysql-spatial/blob/master/composer.json#L18

I think it would be good practice to change for bound php version ; something like :

"require": {
   "php": "^7.4|^8.0",
}

https://getcomposer.org/doc/faqs/why-are-unbound-version-constraints-a-bad-idea.md

[macbookandrew]

There’s a PR open for this: #189