Skip to content
This repository has been archived by the owner on Mar 10, 2020. It is now read-only.

fix: stats/bw uses stream #640

Merged
merged 7 commits into from
Dec 6, 2017
Merged

fix: stats/bw uses stream #640

merged 7 commits into from
Dec 6, 2017

Conversation

hacdias
Copy link
Contributor

@hacdias hacdias commented Dec 6, 2017

No description provided.

@ghost ghost assigned hacdias Dec 6, 2017
@ghost ghost added the in progress label Dec 6, 2017
Copy link
Contributor

@daviddias daviddias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, please add a test :)

@codecov
Copy link

codecov bot commented Dec 6, 2017

Codecov Report

Merging #640 into master will increase coverage by 0.65%.
The diff coverage is 80%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #640      +/-   ##
==========================================
+ Coverage   83.56%   84.22%   +0.65%     
==========================================
  Files         119      119              
  Lines        1771     1775       +4     
==========================================
+ Hits         1480     1495      +15     
+ Misses        291      280      -11
Impacted Files Coverage Δ
src/stats/bw.js 90.9% <80%> (+48.05%) ⬆️
src/stats/repo.js 100% <0%> (+57.14%) ⬆️
src/stats/bitswap.js 100% <0%> (+57.14%) ⬆️

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 a6592dd...0bc8c08. Read the comment docs.

@daviddias daviddias merged commit c4e922e into master Dec 6, 2017
@daviddias daviddias deleted the fix-stats-bw branch December 6, 2017 09:05
@ghost ghost removed the in progress label Dec 6, 2017
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