-
Notifications
You must be signed in to change notification settings - Fork 205
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(cookie-jar): use typeof for checking type of option #158
Conversation
Works around jestjs/jest#2549 when used in Jest
Thanks for the contribution! Before we can merge this, we need @SimenB to sign the Salesforce.com Contributor License Agreement. |
Ping! Can this be merged now? |
is there anything blocking this from merging? @ShivanKaul |
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.
LGTM
LGTM to me too. Shipping 🚢 |
Any ETA to actually release at least as a beta? |
We'll get a release discussion going in our next meeting. We have a few more p/rs we want to bring in, I think. I think you can do some local testing with our master branch by pulling in a GitHub dependency into your package.json. |
Went with patch-package solution for now. |
* chore: prepare for summer20 release (#143) * 0.8.0 * chore: update jest to v25.5.4 (#152) * chore: upgrade jest to 25.5.4 * 0.9.0 (#154) * Update stubs for Summer 20 (#150) * update stubs for Summer 20 * APPLICATION_SCOPE as symbol * 0.9.1 * chore: update yarn lock file Co-authored-by: Aliaksandr Papko <[email protected]> Co-authored-by: Trevor <[email protected]>
Works around jestjs/jest#2549 when used in Jest