yogthos / migratus

MIGRATE ALL THE THINGS!
640 stars 95 forks source link

Cli options #244

Closed sandre1 closed 1 year ago

sandre1 commented 1 year ago

work in progress on migratus https://github.com/yogthos/migratus/issues/240

what i have added so far

The work is in progress, so be gentle :) I need some reviews from you guys if you have the time. Thank you.

yogthos commented 1 year ago

Overall looks good, I added a few small nitpicks, but otherwise I'm happy to merge this in.

sandre1 commented 1 year ago

Hi, thank you for your input, new things learned :). I have also refactored the set-logger-format function to a multy-arity fn. Forward, i want to add logs for some of the migratus core actions. From my side, it is ready to merge, i will make a PR with the other stuff.

yogthos commented 1 year ago

Great, should I push a new version up to Clojars or would you like to wait for the other PR?

sandre1 commented 1 year ago

Great, should I push a new version up to Clojars or would you like to wait for the other PR?

I think it is better to wait for other PR with complete, tested functionality. thank you.

yogthos commented 1 year ago

Sounds like a plan!