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

Update jest to ~21.2.1 #466

Merged
merged 6 commits into from
Jan 5, 2018
Merged

Update jest to ~21.2.1 #466

merged 6 commits into from
Jan 5, 2018

Conversation

nickpape
Copy link
Contributor

@nickpape nickpape commented Jan 4, 2018

No description provided.

@nickpape nickpape requested review from qz2017 and octogonz January 4, 2018 23:22
@iclanton
Copy link
Member

iclanton commented Jan 4, 2018

Approved.

Approved with PullApprove #Resolved

@octogonz
Copy link
Collaborator

octogonz commented Jan 4, 2018

  "dependencies": {

What's all this then? #Resolved


Refers to: common/config/rush/npm-shrinkwrap.json:14 in 2b9abd6. [](commit_id = 2b9abd6, deletion_comment = False)

@@ -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"
Copy link
Contributor

@qz2017 qz2017 Jan 4, 2018

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

Copy link
Contributor Author

@nickpape nickpape Jan 4, 2018

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",
Copy link
Collaborator

@octogonz octogonz Jan 4, 2018

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

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, included


In reply to: 159781970 [](ancestors = 159781970)

Copy link
Collaborator

@octogonz octogonz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🕐

@nickpape
Copy link
Contributor Author

nickpape commented Jan 4, 2018

  "dependencies": {

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)

Copy link
Collaborator

@octogonz octogonz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@octogonz
Copy link
Collaborator

octogonz commented Jan 4, 2018

Please do appropriate testing #Resolved

@nickpape nickpape merged commit b942f54 into master Jan 5, 2018
@nickpape nickpape deleted the nickpape/update-jest branch February 20, 2018 19:46
rikhoffbauer pushed a commit to rikhoffbauer/microsoft__api-documenter that referenced this pull request Mar 23, 2020
* 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
javier-garcia-meteologica pushed a commit to javier-garcia-meteologica/rushstack that referenced this pull request Jun 26, 2020
* 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants