citrusframework / yaks

YAKS is a platform to enable Cloud Native BDD testing on Kubernetes
Apache License 2.0
83 stars 27 forks source link

Get rid of fmt.Print...() methods #398

Open christophd opened 2 years ago

christophd commented 2 years ago

Use proper output writer instead and redirect output

mwmahlberg commented 2 years ago

Do you have any preferences in that regard?

My suggestion would be https://github.com/hashicorp/go-hclog or https://github.com/uber-go/zap

However, it could also make sense to define a logger interface and implement it with a logger of choice. This would provide the advantage that the log implementation could easily be swapped, should the need arise.

mwmahlberg commented 2 years ago

@christophd Your thoughts on this?