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
chore: Update jest packages to v29 #14221
Conversation
Codecov Report
@@ Coverage Diff @@
## main #14221 +/- ##
==========================================
+ Coverage 55.44% 57.55% +2.11%
==========================================
Files 1274 1262 -12
Lines 31859 30689 -1170
Branches 5750 5747 -3
==========================================
Hits 17664 17664
+ Misses 12372 11205 -1167
+ Partials 1823 1820 -3
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
If I remember correctly we add a pbl with v28. Was it fixed with v29? |
@alexandrebodin yes I've mentioned the problem and the fix in the PR description. Everything seems to work now. |
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.
Haven't done added testing but the changes look good
1e9e4a3
to
9aea302
Compare
What does it do?
This updates jest related dependencies to version 29. A previous attempt to update jest@28 didn't work, because we were blocked on a babel-issue that is resolved in v29.
Jest updated the snapshot format slightly, which is why some of them changed: https://jestjs.io/blog/2022/08/25/jest-29
Why is it needed?
Update dependencies.
How to test it?
Jest tests itself.
Related issue(s)/PR(s)