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: Reference Node and MutationObserver off of the window object #77

Merged
merged 1 commit into from
Jul 19, 2018

Conversation

dadamssg
Copy link
Contributor

fix: Reference Node and MutationObserver off of the window object rather than globally

You have to handle shims yourself if not using jest without this

What:
Changes references to Node and MutationObserver to use window rather than globally.

Why:
The polyfills/shims do not work in a non-jest environment

How:
Node -> window.Node
MutationObserver -> window.MutationObserver

Checklist:

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

…her than globally

You have to handle shims yourself if not using jest without this
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.

Thanks!

@kentcdodds kentcdodds merged commit 7730d25 into testing-library:master Jul 19, 2018
@kentcdodds
Copy link
Member

Would you like to add yourself to the contributors table?

@kentcdodds
Copy link
Member

🎉 This PR is included in version 3.0.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@dadamssg
Copy link
Contributor Author

Sure! I felt awkward doing that with such a small change. Should I make another commit on this branch? or create a new one?

@kentcdodds
Copy link
Member

I recommend creating a new branch for every PR.

You may also appreciate this: http://makeapullrequest.com/ :)

@dadamssg
Copy link
Contributor Author

I definitely appreciate that :)

@dadamssg dadamssg mentioned this pull request Jul 19, 2018
4 tasks
alexkrolick added a commit to alexkrolick/dom-testing-library that referenced this pull request Sep 13, 2018
* Update README with new match examples

* Update types

(probably invalid typescript syntax)

* Remove example that messes up TOC

* Fix union type
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.

None yet

2 participants