-
-
Notifications
You must be signed in to change notification settings - Fork 95
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
Improve logging #245
Improve logging #245
Conversation
Added tools.cli as dependency, configured logger
Signed-off-by: Andrei <sandre1>
Overall looks good, I think the only change I'd do would be to default to info when no log level is set. |
Also part of improve logging (sort of :) ) i've dealt with the error in case of no config file provided. |
I think I'm good to merge this in unless there's anything else you were thinking of adding. |
For now, is ok to merge. Thank you. |
sounds like a plan, and let me know when you think is a good time for a release |
work in progress on migratus #240
[1] I have implemented a --verbose CLI option to set the log level.