Skip to content

fixes #44 #74

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

fixes #44 #74

wants to merge 4 commits into from

Conversation

hoschi
Copy link

@hoschi hoschi commented Feb 14, 2018

My try for #44, let me now if something should be different.

@codecov
Copy link

codecov bot commented Feb 14, 2018

Codecov Report

Merging #74 into master will decrease coverage by 4.79%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master      #74     +/-   ##
=========================================
- Coverage   94.38%   89.58%   -4.8%     
=========================================
  Files           3        3             
  Lines          89       96      +7     
  Branches       25       27      +2     
=========================================
+ Hits           84       86      +2     
- Misses          5        8      +3     
- Partials        0        2      +2
Impacted Files Coverage Δ
src/asyncComponent.js 89.74% <100%> (-3.22%) ⬇️
src/createAsyncContext.js 75% <0%> (-25%) ⬇️

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 94fc382...9e4adee. Read the comment docs.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant