Skip to content
This repository has been archived by the owner on Dec 1, 2023. It is now read-only.

Update(hopp): add more misc optimizations (fix #40) #45

Merged
merged 3 commits into from
Jul 4, 2017
Merged

Conversation

karimsa
Copy link
Member

@karimsa karimsa commented Jul 4, 2017

Relevant issues & PRs

See #40.

Checklist

  • This affects the tests of the project.
  • I have updated the tests as needed.

@codecov-io
Copy link

codecov-io commented Jul 4, 2017

Codecov Report

Merging #45 into master will increase coverage by 0.07%.
The diff coverage is 16.03%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #45      +/-   ##
=========================================
+ Coverage     7.8%   7.87%   +0.07%     
=========================================
  Files          58      58              
  Lines        2500    2503       +3     
=========================================
+ Hits          195     197       +2     
- Misses       2305    2306       +1
Impacted Files Coverage Δ
packages/hopp/dist-legacy/fs/index.js 0% <0%> (ø) ⬆️
packages/hopp/dist/streams/bundle.js 0% <0%> (ø) ⬆️
packages/hopp/dist/tasks/mgr.js 0% <0%> (ø) ⬆️
packages/hopp/dist-legacy/hopp.js 0% <0%> (ø) ⬆️
packages/hopp/dist/tasks/loadPlugins.js 0% <0%> (ø) ⬆️
packages/hopp/dist-legacy/tasks/mgr.js 0% <0%> (ø) ⬆️
packages/hopp/dist/cache.js 38.77% <0%> (ø) ⬆️
packages/hopp/dist/hoppfile/load.js 0% <0%> (ø) ⬆️
packages/hopp/dist-legacy/hoppfile/load.js 0% <0%> (ø) ⬆️
packages/hopp/dist/index.js 0% <0%> (ø) ⬆️
... and 12 more

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 50e0875...d5d9a89. Read the comment docs.

@karimsa karimsa merged commit 728f165 into master Jul 4, 2017
@karimsa karimsa deleted the fix_40 branch July 4, 2017 20:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants