-
-
Notifications
You must be signed in to change notification settings - Fork 323
fix: add runtime field of jest config #639
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
Conversation
|
Thanks! Could you please add something to the related fixture so we'll have some coverage? |
package-lock.json
Outdated
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.
This file shouldn't be added
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.
updated
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.
Thanks! I also asked for #639 (comment) - any chance you could work that in as well?
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.
I've missed this comment. Sure, I'll do it
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.
Maybe you've forgot to add the file? It doesn't seem to be in this branch yet.
| testResultsProcessor: 'jest-phabricator', | ||
| snapshotResolver: '<rootDir>/snapshotResolver.js', | ||
| runtime: '@side/jest-runtime', | ||
| }; |
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.
Updated
|
There seems to be a failing test, any chance you could look into it? |
13ba313 to
28f05f0
Compare
|
Is there still interest in wrapping up this pull request? |
d692d08 to
7aa2f6d
Compare
d91d007 to
d4121d9
Compare
|
Closing due to inactivity. |
fix #603