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
Add jest exception #5858
Add jest exception #5858
Conversation
|
Size Report 1Affected ProductsTest Logs |
Size Analysis Report 1This report is too large (1354164 characters) to be displayed here in a GitHub comment. Please use the below link to see the full report on Google Cloud Storage.Test Logs |
packages/firestore/rollup.config.js
Outdated
return ( | ||
"(typeof document === 'undefined' ? new (require('url').URL)" + | ||
"((typeof document === 'undefined' || process.env.JEST_WORKER_ID " + |
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.
Do we need to check whether process
and process.env
are defined?
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.
Yes, that seems important, added.
Hack to fix for Vercel while not re-breaking for Jest.