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

Split logs into different levels #192

Merged

serbrech merged 7 commits into main from karenchen/split-logging-levels

May 8, 2024

Contributor

karenychen commented Jan 3, 2024 •

edited

Loading

Adapt slog and split logs into different levels

github-actions bot commented Jan 3, 2024 •

edited

Loading

Pull Request Test Coverage Report for Build 7403449512

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.0%) to 85.353%

Totals
Change from base Build 7028388512:	1.0%
Covered Lines:	641
Relevant Lines:	751

💛 - Coveralls

karenychen marked this pull request as ready for review

January 4, 2024 18:14

karenychen changed the title ~~[Draft] Split logs into different levels~~ Split logs into different levels

serbrech reviewed

View reviewed changes

v2/logging.go Outdated Show resolved Hide resolved


          switch to use slog

56d165b

karenychen force-pushed the karenchen/split-logging-levels branch from acb0723 to 56d165b Compare

April 26, 2024 00:23

karenychen added 4 commits

April 25, 2024 17:25


          fix print statement

07070b0


          add default slog logger

87fc63f


          log error on managedsettler fail to handle


          update to go1.21

7dcaf38

serbrech reviewed

View reviewed changes

v2/logging.go Outdated Show resolved Hide resolved


          adress comments

2df466a

karenychen requested a review from serbrech

April 29, 2024 18:39

serbrech reviewed

View reviewed changes

v2/logging.go Show resolved Hide resolved

serbrech reviewed

View reviewed changes

v2/logging.go Show resolved Hide resolved

serbrech reviewed

View reviewed changes

v2/logging.go Outdated Show resolved Hide resolved

serbrech reviewed

View reviewed changes

v2/logging.go Show resolved Hide resolved

serbrech reviewed

View reviewed changes

v2/logging.go Outdated Show resolved Hide resolved


          address comments

981158a

serbrech reviewed

View reviewed changes

v2/logging.go Show resolved Hide resolved

serbrech approved these changes

View reviewed changes

serbrech merged commit 7e6033f into main

5 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet