creof / doctrine2-spatial

Doctrine2 multi-platform support for spatial types and functions.
MIT License
272 stars 176 forks source link

Looking for Maintainers! #228

Open sadortun opened 2 months ago

sadortun commented 2 months ago

We would love if you are using this project or can test a PR to contact us bellow or review the PRs so we can merge them.

I personally no longer have a test setup to validate the PRs so I'm a bit reluctant to merge code without some prior validation.

Many people still use this project https://packagist.org/packages/creof/doctrine2-spatial/stats , I'm not sure archiving it would be beneficial overall to the community.

Your help is most welcome!

You can Tag me in any PR that you review when they are ready to be merged.

PowerKiKi commented 2 months ago

The maintainer you are looking for already exists, it is @Alexandre-T. He already did a lot of work, in #209, that was never merged. Meanwhile he created a fork and has been maintaining it for the past 3 years.

In fact, it will soon become more popular than creof/doctrine2-spatial:

image

Instead of trying to revive this project, and splitting the community in two forever, I think it would be wiser (and much easier) to redirect this half of the community to his fork. Don't you think ?

sadortun commented 2 months ago

Alright @Alexandre-T is a hero! 😎

I remember the issue was about potential BC break between 5.3 and 7.1.

That's really unfortunate.

@ddproxy @djlambert any input?

Alexandre-T commented 2 months ago

Thank you @PowerKiKi for your words, thank you @sadortun and @djlambert for your work.

I've ensured compatibility with PHP 8.1+ and all upcoming major versions of doctrine/orm. I've also forked the three parsers: geo-parser, wkt-parser, and wkb-parser. Additionally, I've created comprehensive documentation using Sphinx and hosted it on Read the Docs. To streamline the development process for maintainers, developers, and contributors, I've implemented a one-command setup for a full development stack using docker-compose up -d. Furthermore, the project utilizes GitHub Actions for continuous integration, automatically testing each pull request before merging.

We currently working to be compliant with MariaDB and MSSQL Server.

I read that creof/doctrine-spatial repository is now marked as abandoned. I suggest you to update the four composer.json files to add the below line. So, packagist.org will be automatically updated and developers using your library can immediately find a solution on our repositories. If anyone want to help, you're all very welcomed.

    "abandoned": "longitude-one/doctrine-spatial",
    "abandoned": "longitude-one/wkb-parser",
    "abandoned": "longitude-one/wkt-parser",
    "abandoned": "longitude-one/geo-parser",