Closed AkashRajpurohit closed 5 years ago
Merging #229 into master will not change coverage. The diff coverage is
100%
.
@@ Coverage Diff @@
## master #229 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 88 89 +1
Lines 394 397 +3
=====================================
+ Hits 394 397 +3
Impacted Files | Coverage Δ | |
---|---|---|
src/largest.js | 100% <100%> (ø) |
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 8e5d379...c489ff9. Read the comment docs.
Cool it passed.
Thanks for the suggestion about running the travis-ci script npm run validate
locally to test before commiting. Will keep a habit for doing that in upcoming open source contributions as well. Cheers
Most of the changes in src/index.js is due to prettier configuration of my vscode. everything else is cool!