Closed dichaudhary closed 3 years ago
Merging #46 (5a860e1) into master (c444e25) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #46 +/- ##
=======================================
Coverage 90.12% 90.12%
=======================================
Files 16 16
Lines 1600 1600
Branches 173 173
=======================================
Hits 1442 1442
Misses 151 151
Partials 7 7
Impacted Files | Coverage Δ | |
---|---|---|
...ckages/repository-modernizer/src/util/constants.js | 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 c444e25...5a860e1. Read the comment docs.
isnt this follow up of that issue which we fixed for best buy?
@dichaudhary could you also include a test value in one of the test configs so that this is covered by tests (minor change)?
Thanks, @ManasMaji for reminding, I should have thought of it before! my bad! Anyways the changes are done.
isnt this follow up of that issue which we fixed for best buy?
Not sure @ukgaurav , but I have it seen it with almost all project's configs and have resolved it as part of the issue reported in CQ-4325182
Description
During transformation , the configs like "{0,date,yyyy-MM-dd HH:mm:ss.SSS} {4} [{3}] {5}" for log manager factory was not getting migrated properly.
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: