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

Jest 25 #8862

Merged
merged 7 commits into from
Aug 22, 2019
Merged

Jest 25 #8862

merged 7 commits into from
Aug 22, 2019

Conversation

SimenB
Copy link
Member

@SimenB SimenB commented Aug 22, 2019

Summary

Make sure to use "rebase and merge" rather than squash merging this

For consumers, the only breaking changes should be dropped Node 6 and upgraded JSDOM. I think that's a nice major version, and we can land more of the planned breaking changes in Jest 26. Thoughts on that?

I can put together a small blog post as well.

Note that this branch is available as jest@next on npm.

Test plan

Green CI

@codecov-io
Copy link

Codecov Report

Merging #8862 into master will decrease coverage by 0.07%.
The diff coverage is 40.9%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8862      +/-   ##
==========================================
- Coverage   63.87%   63.79%   -0.08%     
==========================================
  Files         274      274              
  Lines       11627    11591      -36     
  Branches     2854     2842      -12     
==========================================
- Hits         7427     7395      -32     
+ Misses       3569     3566       -3     
+ Partials      631      630       -1
Impacted Files Coverage Δ
packages/jest-config/src/normalize.ts 79.87% <ø> (ø) ⬆️
packages/jest-message-util/src/index.ts 68.34% <ø> (ø) ⬆️
...us/src/legacy-code-todo-rewrite/jestAdapterInit.ts 0% <ø> (ø) ⬆️
packages/jest-jasmine2/src/jasmineAsyncInstall.ts 0% <ø> (ø) ⬆️
packages/jest-environment-node/src/index.ts 48.48% <ø> (ø) ⬆️
packages/jest-util/src/deepCyclicCopy.ts 100% <ø> (+17.64%) ⬆️
packages/jest-runner/src/index.ts 65.95% <ø> (ø) ⬆️
packages/jest-changed-files/src/index.ts 0% <ø> (ø) ⬆️
packages/jest-transform/src/shouldInstrument.ts 88.23% <0%> (ø) ⬆️
packages/jest-runner/src/runTest.ts 2.46% <0%> (+0.03%) ⬆️
... and 11 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b7cb522...cf24070. Read the comment docs.

@SimenB SimenB merged commit 9d5ac83 into master Aug 22, 2019
@SimenB SimenB deleted the next branch August 22, 2019 07:22
haoqunjiang added a commit to haoqunjiang/vue-cli that referenced this pull request Sep 17, 2019
Jest has released version 25 under the `next` dist-tag:
- https://www.npmjs.com/package/jest/v/25.0.0
- jestjs/jest#8862

See breaking changes at
https://github.com/facebook/jest/blob/6c0a16ab2f88ea645c04880a5491b3a8c2dfc66b/CHANGELOG.md

The maintainer promised no more user facing breaking changes planned
jestjs/jest#8956 (comment)

Considering Vue CLI does not bump the major version often, we need to
use the latest possible version.

Currently there will be warnings from ts-jest.
Once this PR gets merged kulshekhar/ts-jest#1232
the warnings will be eliminated.
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants