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

fix(definitions): export correct types for wait and waitForExpect #33

Merged
merged 2 commits into from
Apr 2, 2018

Conversation

jday3
Copy link
Contributor

@jday3 jday3 commented Mar 30, 2018

What: Updates to wait and waitForExpect Typescript definitions

Why: The types are incorrect - wait should not be a default export and waitForExpect was missing arguments

How:

Checklist:

  • Documentation
  • Tests
  • Ready to be merged
  • Added myself to contributors table

@codecov
Copy link

codecov bot commented Mar 30, 2018

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #33   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           5      5           
  Lines          85     85           
  Branches       19     19           
=====================================
  Hits           85     85

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 fd2df8d...b58c8e4. Read the comment docs.

@tsiq-swyx
Copy link
Contributor

just writing to say thank you. i use typescript and this fix is exactly what i need to use it.

@kentcdodds kentcdodds requested a review from lgandecki April 2, 2018 15:01
@kentcdodds
Copy link
Member

Thanks for the review @tsiq-swyx. I'll go ahead and merge this.

@kentcdodds kentcdodds merged commit 2bf760c into testing-library:master Apr 2, 2018
@tsiq-swyx
Copy link
Contributor

mind pushing a patch version to npm for this when you get time? my CI thing doesnt like v1.9.3 because of this thing. no rush at all.

@kentcdodds
Copy link
Member

Hmmm... It should have published a new release automatically. I'm not sure what went wrong. I'll look into it.

kentcdodds pushed a commit that referenced this pull request Apr 2, 2018
There was an issue with a patch release, so this manual-releases.md change is to release a new patch version.

Reference: #33
@kentcdodds
Copy link
Member

Ok, 1.9.4 has been published!

lucbpz pushed a commit to lucbpz/react-testing-library that referenced this pull request Jul 26, 2020
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