-
Notifications
You must be signed in to change notification settings - Fork 186
feat: logger interface added #10
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.
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
feat: logger interface added #10
Conversation
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
ea97f5e to
532c8bd
Compare
532c8bd to
ab539a1
Compare
8311c51 to
c7d53f9
Compare
ab539a1 to
c60da1e
Compare
c60da1e to
dfd98e6
Compare
c7d53f9 to
6f05288
Compare
dfd98e6 to
a187eb8
Compare
a48ced4 to
87d2329
Compare
a187eb8 to
37a6f86
Compare
37a6f86 to
bb49554
Compare
87d2329 to
9af5eb5
Compare
bb49554 to
51ff3e1
Compare
bifrost.go
Outdated
| } | ||
|
|
||
| if logger == nil { | ||
| logger = NewDefaultLogger(interfaces.LogLevelInfo) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how do i change this log level for the default logger while initializing bifrost?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bifrost init takes only logger interface, so if wanna change the log level of the default logger, just pass bifrost.DefaultLogger{level:Info} to the logger interface in Bifrost Init. if nil is passed then it creates a default logger with info level
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
haa but they need to know that they can utilize something called as a default logger, add it to the doc comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes yes
25b5061 to
8b08515
Compare
8b08515 to
12ae714
Compare
12ae714 to
dcc4572
Compare
9af5eb5 to
9493e3f
Compare
9493e3f to
9af5eb5
Compare
dcc4572 to
12ae714
Compare

No description provided.