Skip to content
This repository has been archived by the owner on Aug 11, 2021. It is now read-only.

feature: add getMany() #177

Merged
merged 1 commit into from
Nov 7, 2018
Merged

feature: add getMany() #177

merged 1 commit into from
Nov 7, 2018

Conversation

vmx
Copy link
Member

@vmx vmx commented Nov 7, 2018

With getMany() it's possible to get several blocks with one call.

Closes #132.

@ghost ghost assigned vmx Nov 7, 2018
@ghost ghost added the status/in-progress In progress label Nov 7, 2018
@vmx vmx requested a review from daviddias November 7, 2018 16:08
@ipld ipld deleted a comment from codecov bot Nov 7, 2018
With `getMany()` it's possible to get several blocks with one call.

Closes #132.
@codecov
Copy link

codecov bot commented Nov 7, 2018

Codecov Report

Merging #177 into master will decrease coverage by 0.3%.
The diff coverage is 84.61%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #177      +/-   ##
==========================================
- Coverage   89.36%   89.05%   -0.31%     
==========================================
  Files           1        1              
  Lines         188      201      +13     
==========================================
+ Hits          168      179      +11     
- Misses         20       22       +2
Impacted Files Coverage Δ
src/index.js 89.05% <84.61%> (-0.31%) ⬇️

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 ebd2d1b...5e3b27d. Read the comment docs.

@vmx
Copy link
Member Author

vmx commented Nov 7, 2018

@diasdavid I rebased and added done() to the tests.

@vmx vmx merged commit db7dc8b into master Nov 7, 2018
@ghost ghost removed the status/in-progress In progress label Nov 7, 2018
@vmx vmx deleted the get-many branch November 7, 2018 17:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants