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: Cannot read property watch of undefined in jest v22 version #197

Merged
merged 1 commit into from
Feb 11, 2019

Conversation

xiaoyuhen
Copy link
Contributor

@xiaoyuhen xiaoyuhen commented Feb 8, 2019

Summary

close #195

Test plan

  • Jest-puppeteer works fine in jest v22 version
  • Jest-puppeteer works fine in jest v23 version

@gregberge gregberge merged commit db731a3 into argos-ci:master Feb 11, 2019
@gregberge
Copy link
Member

Thanks!

@xiaoyuhen
Copy link
Contributor Author

@neoziro

should we also publish a 3.9.1 version to resolve this bug? because 4.0.0 version only support jest v24+

@gregberge
Copy link
Member

I don't think so, it doesn't worth it. Migrating to Jest 24 is not a big deal.

@xiaoyuhen
Copy link
Contributor Author

@neoziro

yeah, upgrade Jest is not a big deal, but if we dont plan to support Jest v22, should we update the README to explain this?

@gregberge
Copy link
Member

@xiaoyuhen I think the readme is enough clear about that. We will just drop the line about Jest v22 in one or two months and it will be enough.

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.

Cannot read property 'watch' of undefined
2 participants