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: allow user to define testMatch in package.json (#1067) #1069

Merged
merged 2 commits into from Apr 25, 2018

Conversation

dhensche
Copy link
Contributor

@dhensche dhensche commented Apr 3, 2018

No description provided.

@sylvainar
Copy link

Waiting for it to be merged, nice work!

@dhensche
Copy link
Contributor Author

@laander with this additional commit it will respect options in either location

…errides in either package.json or jest.config.js to function properly

- ninja edit: somehow tests passed even though I used the wrong jest config option in the generator
@dhensche
Copy link
Contributor Author

What is the suggested change for when the tests only fail on CI because of timeouts? just increase the timeouts of the failing tests?

@Ldoppea
Copy link

Ldoppea commented Apr 23, 2018

You should try to push the commit again, the tests are failing on code not-related to your changes and it seems to pass on other merge requests.

Maybe @yyx990803 can help?

@posva
Copy link
Member

posva commented Apr 23, 2018

I relaunched tests

@yyx990803 yyx990803 merged commit cac18f2 into vuejs:dev Apr 25, 2018
@belaad
Copy link

belaad commented Feb 11, 2020

a readme edit would have been great. How do I set the testmatch in package.json?

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.

None yet

7 participants