PostgreSQL-For-Wordpress / postgresql-for-wordpress

A maintained fork of https://wordpress.org/plugins/postgresql-for-wordpress/
GNU General Public License v2.0
232 stars 72 forks source link

Change the extension from txt to md, create image to docker #22

Closed sobrito closed 1 year ago

sobrito commented 2 years ago
kevinoid commented 2 years ago

Thanks for creating the docker image and this PR @sobrito. This project is pretty dead at the moment, so I'm hesitant to take on the maintenance of a Docker image. Is this something you would like to maintain as a separate project? I'd be happy to add a link to your project from the readme.

sobrito commented 2 years ago

We can do it in two ways:

Shall we revive your project together?

kevinoid commented 2 years ago

Thanks @sobrito, that's a very kind offer!

I think maintaining the Docker image as a separate project (your first option) makes sense organizationally.

If you'd also like to help with this project, that would be great! I'll send you an invite shortly. You are welcome to contribute whatever features or bug fixes you'd like.

Don't feel obligated to maintain it. I think the current implementation (which just does a bunch of regular expression substitutions on the SQL - which were written by the previous maintainer/creator) is fragile and unsustainable. There are several known issues in the bug tracker. A better approach might be to parse and transform the SQL, although that has limitations as well. Unfortunately, it would likely require a lot of work to implement. Since I'm not currently using Wordpress, I may not get around to it soon.

mattbucci commented 1 year ago

Thanks for contributing, but I'm closing this PR as I think maintaining a docker image with a wordpress version is a maintenance burden which should be outside the scope of this project.