search

EnterpriseDB / mysql_fdw

PostgreSQL foreign data wrapper for MySQL
Other
521 stars 160 forks source link

Refactor README.md for PGXN #262

Closed mkgrgis closed 1 day ago

mkgrgis commented 1 year ago

For unifying with https://github.com/pgspider FDW documentation template and by https://github.com/ibarwick/firebird_fdw/blob/master/README.md example of advanced FDW documentation.

mkgrgis commented 7 months ago

Ping, my_sql FDW team ? Is the project still active? Are maintainers interesting in unified README.md for PostgreSQL extension network (PGXN) ?

surajkharage19 commented 7 months ago

Hi @mkgrgis,

Yes, this project is active. We will look into the README changes suggested by you.

Thanks.

mkgrgis commented 7 months ago

Hello, @surajkharage19 ! I am ready for review. Also I have similar PR for mongo_fdw https://github.com/EnterpriseDB/mongo_fdw/pull/168 , but this will also stopped without any comments.

mkgrgis commented 5 months ago

Hello, @surajkharage19! My MongoDB README.md refactoring was completely accepted in https://github.com/EnterpriseDB/mongo_fdw/commit/373ea031a0870d105e73c367fdf26fa35cafe797, let's continue our review?

mkgrgis commented 2 months ago

Ping, @surajkharage19 ! Most of other PostgreSQL FDWs yet use documentation structure from this PR: EnterpriseDB mongo_fdw, redis_cluster_fdw, sqlite_fdw, firebird_fdw, influxdb_fdw etc. Why this review was stopped? I think mysql_fdw is very popular and provided by PGDG repository. Have my PR any problems?

mkgrgis commented 3 days ago

Ping @jeevanchalke , @surajkharage19 , @ALL, is this project still active? Does here reviews PRs at all?

surajkharage19 commented 1 day ago

Hi @mkgrgis,

Sorry for the delay in reply. Thanks for submitting the pull request. We have committed this by making a few changes.

mkgrgis commented 1 day ago

Thanks, @surajkharage19. I'll made next PR about README some time later.