-
Notifications
You must be signed in to change notification settings - Fork 724
Remove cleanup references #213
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
Remove cleanup references #213
Conversation
This commit removes any references to the cleanup-after-each file in the react docs as it has been removed. Instructions for skipping auto cleanup were also added to the docs.
This commit removes references to the afterEach function in the react docs and examples
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I love this. Thanks for working on it! Just a few comments.
Co-Authored-By: Kent C. Dodds <[email protected]>
Co-Authored-By: Kent C. Dodds <[email protected]>
Co-Authored-By: Kent C. Dodds <[email protected]>
Co-Authored-By: Kent C. Dodds <[email protected]>
This commit add the cleanup section bac into the api docs for those that may need to do manual cleanups.
Co-Authored-By: Kent C. Dodds <[email protected]>
The api docs now show a relevant example using the AVA testing framework.
…sting-library-docs into remove-cleanup-references
This commit removes the paragraph at the end of the cleanup section in the api docs that talks about configuring the cleanup globally
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
@all-contributors please add @dustinmyers for docs |
I've put up a pull request to add @dustinmyers! 🎉 |
This pull request removes any instances of
cleanup-after-each
andafterEach(cleanup)
in the react docs and examples. From issue #212