Skip to content

[#78] Updated readme to add wait best practice #79

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

Conversation

RyanAtViceSoftware
Copy link
Contributor

fixes #78

What:
readme.md was updated to give the advice that @kentcdodds gave me in that issue around best practices for using wait.

Why:
I'm hoping to save other folks time ramping up on how to create valuable tests.

How:

Checklist:

  • Documentation
  • Tests N/A
  • Ready to be merged
  • Added myself to contributors table

@gnapse
Copy link
Member

gnapse commented May 7, 2018

I think you should exclude all those .idea/* files off the PR. You should probably have a global .gitignore for that kind of things.

@RyanAtViceSoftware RyanAtViceSoftware force-pushed the 78-updated-readme-to-add-wait-best-practice branch from a6f5473 to 686a58c Compare May 7, 2018 18:51
@RyanAtViceSoftware
Copy link
Contributor Author

@gnapse updated to remove WebStorm files. Thanks for the .gitignore tip!

Copy link
Member

@kentcdodds kentcdodds left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is perfect. Thank you!

@kentcdodds kentcdodds merged commit 1a167a0 into testing-library:master May 7, 2018
@kentcdodds
Copy link
Member

🎉 This PR is included in version 3.0.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

julienw pushed a commit to julienw/react-testing-library that referenced this pull request Dec 20, 2018
* docs: added Using Without Jest section

* use jsdom-global

I discovered a package that can simplify this process greatly :)
lucbpz pushed a commit to lucbpz/react-testing-library that referenced this pull request Jul 26, 2020
chore: 🤖 update react-testing-library
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.

Tests don't fail gracefully when using wait
3 participants