Closed sweezy793 closed 5 years ago
Merging #48 into master will not change coverage. The diff coverage is
100%
.
@@ Coverage Diff @@
## master #48 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 37 38 +1
Lines 354 369 +15
=====================================
+ Hits 354 369 +15
Impacted Files | Coverage Δ | |
---|---|---|
src/index.js | 100% <100%> (ø) |
:arrow_up: |
src/padovan/index.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 96c2c7a...dc654fb. Read the comment docs.
@sweezy793 I've resolved the conflicts for this PR but from next time onwards please do this by yourself and please do not alter the single quotes by the double quotes as that was the reason for the merge conflict.
Yes, I will keep this in mind next time, Actually I had kept it ad single quotes but prettier changed it into double quotes.
Do the checklist before submitting the PR:
Q: What version of Node.js you've used in the project? (
e.g. Node v10.11.0
) A: v10.15.1Q: Mention the Issue Number! (
e.g. Fixed #8
) A: #46Give additional information regarding the PR below: Added padovan sequence in src by taking reference from fibonacci and required it in src/index.js