-
Notifications
You must be signed in to change notification settings - Fork 0
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
pruned deprecated code and code that shouldn't be maintained any longer #43
Conversation
Kudos, SonarCloud Quality Gate passed!
|
Codecov Report
@@ Coverage Diff @@
## main #43 +/- ##
==========================================
+ Coverage 98.93% 99.35% +0.42%
==========================================
Files 17 12 -5
Lines 657 467 -190
==========================================
- Hits 650 464 -186
+ Misses 6 2 -4
Partials 1 1
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
|
||
// GetDefaultLogger returns the default logger, which doesn't do anything. | ||
func GetDefaultLogger(_ context.Context) *zap.Logger { | ||
return Default() | ||
} | ||
|
||
// GetNilLogger returns nil. | ||
func GetNilLogger(_ context.Context) *zap.Logger { | ||
return nil | ||
} |
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.
cool, I just removed these references from other servers
// GetLoggerFunc is the function used to get a request-specific logger from | ||
// its context. | ||
type GetLoggerFunc func(context.Context) *zap.Logger | ||
|
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.
cool, I just removed this type from servers that were using it
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.
lgtm! 🍻
This prunes out code we don't want any longer and code that duplicates existing functionality.