moved aws-rollout script to logger instead of printing#8
moved aws-rollout script to logger instead of printing#8akhon wants to merge 1 commit intoDevOps-Nirvana:masterfrom
Conversation
|
@akhon Hey thanks for the pull request. Question: Do you think it would make more sense to actually use the various log levels in here? I believe I see you only used This may be an opportunity to make this script less verbose by default, and have only the critical pieces of info logged (under .info) and move some things over into .warn and/or into .debug or .error, etc. That could make the usage of this in CI/CD systems a bit less (overly) verbose. Would you like to take an attempt at doing this? I'd be happy to review and try your branch for one of my customers as well and see how I like it. |
please take a look. this is more powerful logging tool rather than plain printing