Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add MySql support #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add MySql support #9

wants to merge 2 commits into from

Conversation

Ranger2959
Copy link

After reading a bunch about StackOverflow's DB migration process, I started writing one based on the blog posts I could find, until I found out about this repo. I'm mainly using MySql in my projects now, so I wanted a version of this migrator for MySql, so that's what this pull request is.

Let me know what you think and i'm open to any suggestions you have about the implementation.

@Ranger2959 Ranger2959 changed the title Add my sql support Add MySql support Jul 4, 2017
@bretcope
Copy link
Owner

Hey, I haven't meant to ignore this for so long. I just have no time to maintain this library at the moment. I started a complete redesign at one point because I wasn't particularly happy with the initial design, but I ran out of free time. It's different enough that if I ever finish it, I probably shouldn't even call it Mayflower.

So, in essence, I'm saying I don't have time to work on this library, and if I did, I'd be working on a replacement, instead of improving the existing code. If you're interested enough, I could make you a maintainer.

At a glance, your changes look good, except for how you deleted the sample SQL Server files. There's also a part of me that feels like a solution which was extensible, and would result in different libraries for different db providers would probably be better than pulling every database in as a dependency. However, that is also more work, and I don't have the time to care much at this point.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants