mysql2postgres icon indicating copy to clipboard operation
mysql2postgres copied to clipboard

update for pg 0.11.0 and a few minor changes

Open tardate opened this issue 14 years ago • 9 comments

Hi Max, I've integrated a few contributions to the project, updated some of the docs and stepped the gem version to 0.1.1. All tests green at my end!

tardate avatar May 02 '11 22:05 tardate

Hi Max, are you still maintaining this gem?

I just spent a bit of time today merging the changes from a bunch of people: James Coleman, Aaron Peckham, James Tippett, Tim Morgan", dakhota, Matthew Soldo.

Changes are all on this pull request now (which has also merged you own master, so it should be an easy fast-forward merge).

Tests are all green - hopefully that means I got everyone's changes in OK(!)

tardate avatar Feb 12 '12 20:02 tardate

Hi, can you, please, rebase your updates so, that I merge them?

maxlapshin avatar Jul 19 '12 19:07 maxlapshin

Hi Max, sure I'll take another look now. Hopefully we can get the number of independent forks back to something a little more reasonable..

tardate avatar Jul 20 '12 18:07 tardate

hmm, so I took a look at the latest master and its not a very clean merge. various people have been heading in different directions.

a few things that need to be resolved, and they require an opinion:

I'm not so sure about the changes made to use config/database.yml by default, and drop the ability of the tool to generate a config template for the user. Fine if you are running the tool from a cloned repo, but not consistent with using the tool as a gem.

There's also an unresolved issue of who's managing the gemspec. Current master is along the lines of manually managed, but jeweler is still in the project and Rakefile. Also, version in the gemspec is not consistent with the version that the gem reports.

Why have we removed all of the good info in the readme?

Max, you have any thoughts on which way to go on these issues?

tardate avatar Jul 20 '12 20:07 tardate

I think you can do anything you consider right =)

maxlapshin avatar Jul 20 '12 20:07 maxlapshin

ok;-)

I'll get a fully resolved merge first, then look at some changes on top to clean up a bit

tardate avatar Jul 20 '12 20:07 tardate

Any updates on these changes?

paazmaya avatar Sep 15 '14 15:09 paazmaya

We can’t automatically merge this pull request.

maxlapshin avatar Sep 15 '14 15:09 maxlapshin

Ok, well let's see if I could sort these out...

paazmaya avatar Sep 16 '14 15:09 paazmaya