-
Notifications
You must be signed in to change notification settings - Fork 23
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
Removing un-needed packages #1
Conversation
This PR has merge conflicts |
Just some merge conflicts... sorry for the long response |
package.json
Outdated
"pify": "3.0.0", | ||
"throat": "4.1.0", | ||
"worker-farm": "1.5.0" | ||
"jest-worker": "^21.2.1" |
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.
can you bump this to 22? I need jestjs/jest#5068 :P
e9b31f2
to
dcd6421
Compare
Rebased this, and bumped to jest-worker@22 |
@@ -2,8 +2,8 @@ | |||
|
|||
exports[`Works when it has failing tests 1`] = ` | |||
"FAIL integrationTests/__fixtures__/failing/__src__/file1.js | |||
Company policies require ⚔️ 🏃 in every file |
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.
not sure about this one. Change in jest 22
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.
Not sure that I understand why that happened
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.
It's a deliberate change in jest 22
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.
Tracked this down, it's jestjs/jest#4504 (which was released in jest 21.2.0)
d099239
to
4458f8c
Compare
7e43e8f
to
74926fb
Compare
74926fb
to
8fcb8e1
Compare
I've left jest at version 21 for the node 4 support. Can drop it in a separate PR if wanted |
I noticed this package was significantly larger than I was expecting, and it seems like there's a bunch of prod dependencies that aren't needed. This just cleans them up.