Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Change type of cleanup function #23

Merged
merged 2 commits into from
Jun 13, 2020

Conversation

denis-ok
Copy link
Contributor

@denis-ok denis-ok commented Jun 8, 2020

Return type for cleanup function changed to Js.Promise.t(unit) according to source:
https://github.com/testing-library/react-testing-library/blob/6833b8a0a78817782a0c4df3d8e33271c688e93e/src/pure.js#L99

@denis-ok denis-ok force-pushed the Change-type-of-cleanup-function branch 2 times, most recently from cb4c4b0 to 630a989 Compare June 8, 2020 20:20
@denis-ok denis-ok force-pushed the Change-type-of-cleanup-function branch 6 times, most recently from 18c573e to 3c5f326 Compare June 10, 2020 08:17
@codecov-commenter
Copy link

codecov-commenter commented Jun 10, 2020

Codecov Report

Merging #23 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #23   +/-   ##
=======================================
  Coverage   98.90%   98.90%           
=======================================
  Files           1        1           
  Lines          91       91           
=======================================
  Hits           90       90           
  Misses          1        1           
Impacted Files Coverage Δ
src/ReactTestingLibrary.re 98.90% <ø> (ø)

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 12e7874...a864ee3. Read the comment docs.

@denis-ok denis-ok force-pushed the Change-type-of-cleanup-function branch from 3c5f326 to a864ee3 Compare June 10, 2020 08:19
@wyze wyze merged commit 1e4cd6c into wyze:master Jun 13, 2020
@wyze
Copy link
Owner

wyze commented Jun 13, 2020

Looks good, thank you!

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

Successfully merging this pull request may close these issues.

3 participants