Laravel 10#206

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Open

limenet wants to merge10 commits intogrimzy:master

base:master

Choose a base branch

fromlimenet:laravel-10

Open

Laravel 10#206

limenet wants to merge10 commits intogrimzy:masterfromlimenet:laravel-10

Conversation

Copy link

asanikovich commentedJun 2, 2023

@limenet try a look athttps://github.com/asanikovich/laravel-spatial

Copy link

OzanKurt commentedJul 6, 2023

@limenet Can you please publish this pull request as a new package?

This package is not being maintained since 2020.

Copy link

Author

@OzanKurt Normally I'm hesitant to take over a package but as you mentioned, this package appears to be abandoned, so there's a new repository now over athttps://github.com/limenet/laravel-mysql-spatial.

Before I tag a release, could you please give it a try?

Seehttps://github.com/limenet/laravel-mysql-spatial#migration-from-grimzylaravel-mysql-spatial for a (very) short migration guide
Please install the package usingcomposer require limenet/laravel-mysql-spatial:dev-master for now

If you find any issues, please let me knowby opening an issue (or even a pull request!). Thank you!

Compared to this PR, I made a few more changes that shouldn't affect functionality (except for the namespace change mentioned in the README):

Changed namespace fromGrimzy\ toLimenet\
PHPStan level increased from 6 to 8
Suppressed a deprecation notice
Used rector to upgrade code

Copy link

OzanKurt commentedJul 7, 2023

@limenet I've alsi copied to a custom package for personal use, I've been using some of the functionality that the package offers. I've not had any problems so far.

Your readme looks amazing!