yogthos / migratus

MIGRATE ALL THE THINGS!
651 stars 95 forks source link

Improve logging #245

Closed sandre1 closed 1 year ago

sandre1 commented 1 year ago

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

[1] I have implemented a --verbose CLI option to set the log level.

yogthos commented 1 year ago

Overall looks good, I think the only change I'd do would be to default to info when no log level is set.

sandre1 commented 1 year ago

Also part of improve logging (sort of :) ) i've dealt with the error in case of no config file provided.

yogthos commented 1 year ago

I think I'm good to merge this in unless there's anything else you were thinking of adding.

sandre1 commented 1 year ago

For now, is ok to merge. Thank you. I will continue work on a new branch.

yogthos commented 1 year ago

sounds like a plan, and let me know when you think is a good time for a release