Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing it.
Nit: the logger.Logger wording feels too verbose to use everywhere in the project (not sure why Linter is happy with it 😬😅).

I think either we dot import (not a big fan either 😅 , but don't mind)
Or we could have the log level functions (as public functions) in the cli logger package, with the same function signature as the logrus lib, so that we could directly call logger.Infof (Same as what logrus global log does).
WDYT?