Conversation

joual

What:

Why:

How:

Checklist:

  • Documentation added to the
    docs site
  • Tests
  • Typescript definitions updated
  • Ready to be merged

Closes #345
Closes #347

kentcdodds
kentcdodds previously approved these changes Apr 5, 2019
alexkrolick
alexkrolick previously approved these changes Apr 5, 2019
@kentcdodds

This actually isn't enough to fix it for 16.9.0 unfortunately. I'll go ahead and pull this PR and work on it a bit more. Thanks!

@joual

Ya I figured, and you might want to put the test somewhere that makes more sense or massage it into something that might detect all the cases you need :)

@joual

Thank you for all your work!

@joualjoual closed this Apr 5, 2019
@kentcdodds

Oh, let's keep this open. I'll just update your PR :)

@kentcdoddskentcdodds reopened this Apr 5, 2019
@kentcdoddskentcdodds dismissed stale reviews from alexkrolick and themself via c660280 April 5, 2019 19:40
@kentcdodds

@allcontributors[bot], please add @joual for bug, test, and code

@allcontributors

@kentcdodds

I've put up a pull request to add @joual! 🎉

@kentcdoddskentcdodds merged commit 021ef9d into testing-library:master Apr 5, 2019
@joualjoual deleted the failed-find-by branch April 5, 2019 19:46
@kentcdodds

🎉 This PR is included in version 6.1.2 🎉

The release is available on:

  • npm package (@latest dist-tag)
  • release

Your semantic-release bot 📦🚀

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 . Already have an account? Sign in to comment
None yet