Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Use logger instead of fmt.Println() #251

Open
artursouza opened this issue Apr 19, 2023 · 1 comment
Open

Use logger instead of fmt.Println() #251

artursouza opened this issue Apr 19, 2023 · 1 comment
Labels
good first issue Good for newcomers kind/bug

Comments

@artursouza
Copy link
Member

Expected Behavior

No Println() statement in code, all logs use proper logger with log level.

Actual Behavior

Println() everywhere :)

Steps to Reproduce the Problem

Release Note

RELEASE NOTE: FIX log messages to use proper log levels and not Println()
@artursouza artursouza mentioned this issue Apr 19, 2023
Merged
@ShatilKhan
Copy link

I this issue open for contribution @artursouza ?
If so I'd like to take it on

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers kind/bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@artursouza @ShatilKhan