Skip to content

Static logger API#513

Merged
dmdashenkov merged 3 commits intomasterfrom
static-logger-api
Feb 25, 2020
Merged

Static logger API#513
dmdashenkov merged 3 commits intomasterfrom
static-logger-api

Conversation

@dmdashenkov
Copy link
Contributor

In this PR we add the Logging.loggerFor(Class) method which obtains a logger for a given class without making the class implement Logging interface.

@dmdashenkov dmdashenkov self-assigned this Feb 25, 2020
@dmdashenkov dmdashenkov requested a review from armiol February 25, 2020 14:46
@dmdashenkov
Copy link
Contributor Author

@armiol, PTAL.

@codecov
Copy link

codecov bot commented Feb 25, 2020

Codecov Report

Merging #513 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #513      +/-   ##
============================================
+ Coverage     73.54%   73.55%   +<.01%     
- Complexity     2880     2881       +1     
============================================
  Files           500      500              
  Lines         11693    11695       +2     
  Branches        650      650              
============================================
+ Hits           8600     8602       +2     
  Misses         2866     2866              
  Partials        227      227

@dmdashenkov dmdashenkov merged commit ae32eaf into master Feb 25, 2020
@dmdashenkov dmdashenkov deleted the static-logger-api branch February 25, 2020 15:49
@armiol armiol mentioned this pull request Mar 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants