Closed leonm1 closed 4 years ago
Thanks for the review, @cktang88 and the suggestions, @aadibajpai!
I will wait for another day or two for @gfting and @bencooper222 to review to see if they have anything to add.
Merging #714 into main will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## main #714 +/- ##
=======================================
Coverage 39.52% 39.52%
=======================================
Files 97 97
Lines 2760 2760
Branches 627 627
=======================================
Hits 1091 1091
Misses 1669 1669
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 cb2d069...b3ffcf2. Read the comment docs.
Code Climate has analyzed commit b3ffcf24 and detected 0 issues on this pull request.
View more on Code Climate.
CLRF -> LF