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

globalThis #60

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

globalThis #60

wants to merge 6 commits into from

Conversation

brettz9
Copy link

@brettz9 brettz9 commented May 14, 2021

Builds on #55, #56, #57, #58, and #59.

Grabbing globalThis allows working in Node as well as in workers.

Fixes #46.

To get full coverage within the constructor (when `now` is `null`), separate tests which have `set` in `beforeEach`

Also adds source maps (to properly trace as well as user benefit)
@brettz9 brettz9 mentioned this pull request May 14, 2021
Copy link

@long-lazuli long-lazuli left a comment

Choose a reason for hiding this comment

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

This PR save our day.

yarn add mockdata#91d2350415a9277869eec25077647187cde0e16c

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.

'ReferenceError: Date is not defined' with Babel (Includes fix)
2 participants