-
Notifications
You must be signed in to change notification settings - Fork 2k
Testing docs #2665
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
Testing docs #2665
Conversation
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.
Looks great, thank you for the PR!
|
||
### Libraries | ||
|
||
#### With create-react-app |
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 think you can actually delete this section on using create-react-app now - the libraries ship with CJS as the default module format
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.
Sure thing.
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.
Oh! I think I'm trying to push the same changes after you and that's why it's getting rejected!
Thanks for the speedy review and tidies for the spellcheck ignore.
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.
no problem, thank you for the update!
The build failure looks like it's throwing on the |
* docs: spelling updates, remove CRA instructions * docs: add DevTools to spelling * add redux devtools config info * re-order + add JSDOM related testing info Co-authored-by: Chris Trevino <[email protected]>
Edit Project Documentation for Testing + Troubleshooting
Description
Rationale
I just spent a little too long getting this to work, looking up the DataTransfer object, reading the React DND source code, and wondering if my problem was that there wasn't a DragEvent in JSDOM.
Turns out it was simple... if you do it right 😝