Thanks to visit codestin.com
Credit goes to github.com

Skip to content

chore: bump jest to version 24 #159

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

Merged
merged 6 commits into from
Feb 19, 2019
Merged

chore: bump jest to version 24 #159

merged 6 commits into from
Feb 19, 2019

Conversation

JamesHenry
Copy link
Member

This allows me to get rid of my temporary override in the integration test setup.

We will merge this after kulshekhar/ts-jest#957 goes in and new release of ts-jest is available. Right now, everything works fine, but ts-jest produces a lot of noise because of the version mismatch.

@JamesHenry JamesHenry added the blocked by another PR PRs which are ready to go but waiting on another PR label Jan 29, 2019
@codecov
Copy link

codecov bot commented Jan 29, 2019

Codecov Report

Merging #159 into master will increase coverage by 1.55%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #159      +/-   ##
==========================================
+ Coverage    95.2%   96.76%   +1.55%     
==========================================
  Files          67       67              
  Lines        2797     2350     -447     
  Branches      721      335     -386     
==========================================
- Hits         2663     2274     -389     
  Misses         51       51              
+ Partials       83       25      -58
Impacted Files Coverage Δ
packages/eslint-plugin/src/util/misc.ts 80% <0%> (-1.82%) ⬇️
...eslint-plugin/src/rules/no-parameter-properties.ts 92.85% <0%> (-1.59%) ⬇️
packages/eslint-plugin/src/rules/indent.ts 90.9% <0%> (-0.76%) ⬇️
.../eslint-plugin/src/rules/restrict-plus-operands.ts 95.45% <0%> (-0.7%) ⬇️
packages/eslint-plugin/src/util/deepMerge.ts 93.75% <0%> (-0.37%) ⬇️
packages/typescript-estree/src/ast-converter.ts 100% <0%> (ø) ⬆️
...int-plugin/src/rules/require-array-sort-compare.ts 100% <0%> (ø) ⬆️
...lint-plugin/src/rules/no-triple-slash-reference.ts 100% <0%> (ø) ⬆️
.../eslint-plugin/src/rules/no-useless-constructor.ts 100% <0%> (ø) ⬆️
packages/parser/src/parser.ts 100% <0%> (ø) ⬆️
... and 42 more

@bradzacher bradzacher added the tests anything to do with testing label Feb 12, 2019
@armano2 armano2 removed the blocked by another PR PRs which are ready to go but waiting on another PR label Feb 19, 2019
@JamesHenry JamesHenry merged commit 6b643c3 into master Feb 19, 2019
@JamesHenry JamesHenry deleted the jest24 branch February 19, 2019 13:07
kaicataldo pushed a commit to kaicataldo/typescript-eslint that referenced this pull request Aug 27, 2019
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
tests anything to do with testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants