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

fix(jest): update usage of jest.runCli #897

Merged
merged 1 commit into from
Jul 2, 2018

Conversation

1hella
Copy link
Contributor

@1hella 1hella commented Jun 26, 2018

jest.runCli returns a promise. This change updates the usage of jest.runCli to use the promise rather than pass in a callback which never will get executed.

closes #896

jest.runCli returns a promise. This change updates the usage of jest.runCli to use the promise rather than pass in a callback which never will get executed.

closes aurelia#896
@CLAassistant
Copy link

CLAassistant commented Jun 26, 2018

CLA assistant check
All committers have signed the CLA.

@Alexander-Taran
Copy link
Contributor

@1hella which version of jest breaks behavior?
Was it a major bump? Because this kind of change will be backwards incompatible. So the jest dependency should be changed as well.

@1hella
Copy link
Contributor Author

1hella commented Jun 27, 2018

@Alexander-Taran I'll double check that

@1hella
Copy link
Contributor Author

1hella commented Jun 28, 2018

@Alexander-Taran It looks like this change was added in jest-cli version 21.0.0, and the version in the package.json generated from au new looks to be ^23.0.0-alpha.0, so shouldn't need to be changed.

@JeroenVinke JeroenVinke merged commit fc66cbf into aurelia:master Jul 2, 2018
@JeroenVinke
Copy link
Collaborator

Thank you!

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.

jest.runCli returns a promise now, the callback in tasks/jest.js never gets executed
4 participants