Skip to content

Conversation

@fjcaetano
Copy link
Owner

Adds generic types overloads for useEffectUpdate and useConditionalEffect

fix #14

@codecov-io
Copy link

Codecov Report

Merging #15 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #15   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines          56     56           
  Branches        5      5           
=====================================
  Hits           56     56
Impacted Files Coverage Δ
src/index.tsx 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 37ad771...a6955ca. Read the comment docs.

@fjcaetano fjcaetano merged commit a67b9c0 into master Nov 25, 2019
@fjcaetano fjcaetano deleted the fix/use-effect-update-types branch November 25, 2019 20:16
@fjcaetano fjcaetano added this to the 1.0.0 milestone Nov 25, 2019
@fjcaetano fjcaetano added the bug Something isn't working label Nov 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] useEffectUpdate has wrong types for previous values

3 participants