-
Notifications
You must be signed in to change notification settings - Fork 451
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
feat: support Jest 25 v2 #1355
feat: support Jest 25 v2 #1355
Conversation
…st25 � Conflicts: � e2e/__templates__/default/package-lock.json � e2e/__templates__/default/package.json � e2e/__templates__/with-babel-7/package-lock.json � e2e/__templates__/with-babel-7/package.json � e2e/__templates__/with-typescript-2-7/package-lock.json � e2e/__templates__/with-typescript-2-7/package.json � e2e/__templates__/with-unsupported-version/package-lock.json � e2e/__templates__/with-unsupported-version/package.json � package-lock.json
@kulshekhar Any ideas why deep-path.test.ts would be failing? |
hi @kibertoad , would you please also including update e2e template into this PR ? You can use command updated Sorry my bad I saw e2e templates were included already. |
@kibertoad I'm not sure. I'll take a look once the CI run is done to see if there's anything I can identify offhand |
@kibertoad I think, in the failing tests, the snapshots just have to be regenerated |
Pull Request Test Coverage Report for Build 3889
💛 - Coveralls |
@kulshekhar It's green now! |
package.json
Outdated
@@ -69,7 +70,7 @@ | |||
"yargs-parser": "10.x" | |||
}, | |||
"peerDependencies": { | |||
"jest": ">=24 <25" | |||
"jest": ">=24 <26" |
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.
Change this to ">=25 <26"
src/util/version-checkers.ts
Outdated
TypeScript = '>=2.7 <4', | ||
BabelJest = '>=24 <25', | ||
BabelJest = '>=24 <26', |
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.
Change this to ">=25 <26"
src/util/version-checkers.ts
Outdated
@@ -10,9 +10,9 @@ const logger = rootLogger.child({ namespace: 'versions' }) | |||
* @internal | |||
*/ | |||
export enum ExpectedVersions { | |||
Jest = '>=24 <25', | |||
Jest = '>=24 <26', |
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.
Change this to ">=25 <26"
@kibertoad thanks! Just added a request for a few minor changes |
@kulshekhar Addressed! |
@kulshekhar Aaaaand it's green again. |
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.
@kibertoad @sodatea thanks for your work! I'll have a release out shortly
Based on #1232
fixes #1354