Closed rajarju closed 4 years ago
Merging #25 into master will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #25 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 3 3
Lines 89 90 +1
Branches 12 12
=========================================
+ Hits 89 90 +1
Impacted Files | Coverage Δ | |
---|---|---|
src/AioLogger.js | 100.00% <100.00%> (ø) |
|
src/DebugLogger.js | 100.00% <100.00%> (ø) |
|
src/WinstonLogger.js | 100.00% <100.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 9610165...a2a7919. Read the comment docs.
@moritzraho
thanks for doing this @rajarju got annoyed by this multiple times 😄 lgtm just maybe missing a last one here https://github.com/adobe/aio-lib-core-logging/blob/rajarju/string-interpolation/src/DebugLogger.js#L90?
I guess I did miss this, May be will add tests for other formats too
lgtm !
Description
Adding support for string substitutions in AioLogger Both debug and winston logger will support string substitution
Related Issue
Fix: #20
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: