Skip to content
This repository has been archived by the owner on Jun 28, 2018. It is now read-only.

Future of migrate #311

Open
mattes opened this issue Nov 18, 2017 · 12 comments
Open

Future of migrate #311

mattes opened this issue Nov 18, 2017 · 12 comments

Comments

@mattes
Copy link
Owner

mattes commented Nov 18, 2017

Everyone,

I started migrate 3 years ago and I'm happy it's seeing widespread adoption. I see forks at @99designs, @codeship, @uber, @segmentio, @newrelic, @gitGNU, @postmates and many more who find migrate helpful and try to help pushing it forward.

Some months ago I co-founded www.templarbit.com and ever since things are busier than ever. It's unfortunate, but I won't be able to keep maintaining migrate.

Who is interested in stepping in and taking migrate over?

@jadbox
Copy link

jadbox commented Nov 23, 2017

I just discovered this tool.. and sad that it's already going to lose its author.

Would you be willing to move the repo to its own org (maybe 'gomigrate') and allow active contributors to have commit access? It's probably the best way to keep the project maintained by giving ownership over to the lead contributors.

@dhui
Copy link

dhui commented Jan 19, 2018

I've created a fork with the following goals:

  1. Get tests passing again
  2. Merge in existing open PRs that add value (this is subjective) and require very little effort to verify correctness
  3. Push a new release

I won't have much time to dedicate to maintaining this fork so I'm looking for co-maintainers as well.

@dropwhile
Copy link

dropwhile commented Jan 19, 2018

@dhui looks like the build is broken due to cockroachdb not supporting go1.7 (it apparently uses a few things that apparently were added to go1.8).

# github.com/cockroachdb/cockroach-go/crdb
../../cockroachdb/cockroach-go/crdb/tx.go:41: undefined: sql.TxOptions
../../cockroachdb/cockroach-go/crdb/tx.go:43: db.BeginTx undefined (type *sql.DB has no field or method BeginTx)

This is the one that broke, apparently.
cockroachdb/cockroach-go#41 (comment)

So probably need to either (a) drop go 1.7 support (b) vendor/pin cockroachdb to before that was merged

@dhui
Copy link

dhui commented Jan 21, 2018

FYI, the fork is now ready to use!

https://github.com/golang-migrate/migrate/releases/tag/v3.1.0

Tests are passing again, so contributions are welcome.

@dcormier
Copy link

It would be nice to see additional committers in that fork to avoid this happening again, @dhui.

@dhui
Copy link

dhui commented Jan 26, 2018

@dcormier

It would be nice to see additional committers in that fork to avoid this happening again, @dhui.

Agreed, but that's beyond my control. Technically you're a committer since I cherry-picked your commit 😉

Maybe we'll have another PR merged in after this one was closed: golang-migrate/migrate#4

@dcormier
Copy link

Technically you're a committer since I cherry-picked your commit

I guess I mean someone else (anyone) who is responsive and has control of the org, in case you're kidnapped by pygmies.

@avelino
Copy link

avelino commented Feb 4, 2018

@mattes and maintainer team
We use a lot of the migrate in pREST[1], if you want to move the project to the organization of pREST[2] will be very welcome (we have a great team to keep it)

[1] https://github.com/prest/prest
[2] https://github.com/prest

@dhui
Copy link

dhui commented Feb 21, 2018

@avelino I'm hesitant to move the repo again unless there's a good reason to do so.

You and your team should feel free to make contributions to the fork. I can also make you and your team collaborators if you so desire once you've made a few contributions.

@mixedCase
Copy link

@mattes Could a fork be blessed as the official continuation and this repo marked in the README as abandoned? I vendored it and was only aware it had been abandoned because I needed the Cassandra driver and it wasn't available in the latest release.

And now's a good a time as any: Thank you for all the work you did on migrate, it has been my go-to choice for Go projects for a while and certainly proved useful. Best of luck in your new endeavor!

@Fire-Dragon-DoL
Copy link

Thanks I would like this to be made official. We started using the tool today, it's fantastic being able to move out any DB migration out of the main app (much easier to deploy!)

@nfisher
Copy link

nfisher commented Mar 14, 2018

@dhui what're your thoughts on just adding @avelino and @dcormier as admins? Worst case they do nothing... best case they help a little. :)

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

No branches or pull requests

9 participants