-
Notifications
You must be signed in to change notification settings - Fork 608
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
Update jest to ~21.2.1 #466
Conversation
@@ -29,6 +29,6 @@ | |||
"@types/js-yaml": "3.9.1", | |||
"@types/node": "6.0.88", | |||
"gulp": "~3.9.1", | |||
"@types/jest": "~20.0.4" | |||
"@types/jest": "~21.1.8" |
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.
Did you test the case when one project fails to build? #Resolved
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.
Yes! #Resolved
@@ -11,7 +11,7 @@ | |||
"devDependencies": { | |||
"@types/chai": "3.4.34", | |||
"chai": "~3.5.0", | |||
"@types/jest": "~20.0.4", | |||
"@types/jest": "~21.1.8", |
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.
"~21.1.8" [](start = 19, length = 9)
do you need the PNPM workaround in this PR? #Resolved
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.
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.
🕐
The old version of gulp-core-build is apparently using the older jest. Once this is published, the next PR will upgrade the cyclic version of GCB that we are using to be the one we just published, which should make this go away In reply to: 355429219 [](ancestors = 355429219) Refers to: common/config/rush/npm-shrinkwrap.json:14 in 2b9abd6. [](commit_id = 2b9abd6, deletion_comment = False) |
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.
Please do appropriate testing #Resolved |
* Update jest, jest-cli, and @types/jest * Rush Generate * Update for changes to Jest API * Changefiles * Add monkey patch for symlinks in jest-resolve * Refactor for readability
* Update jest, jest-cli, and @types/jest * Rush Generate * Update for changes to Jest API * Changefiles * Add monkey patch for symlinks in jest-resolve * Refactor for readability
No description provided.