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

Run test on Windows #38

Merged
merged 1 commit into from
Dec 25, 2017
Merged

Run test on Windows #38

merged 1 commit into from
Dec 25, 2017

Conversation

feugy
Copy link
Contributor

@feugy feugy commented Dec 22, 2017

Hello!

I'd like to give a try on instrumenting assert.throws() as requested here.
While digging into the power-assert galaxy, I realised I couldn't run the tests on a Windows OS.

This PR makes it possible to run all tests on such environment.

@coveralls
Copy link

coveralls commented Dec 22, 2017

Coverage Status

Coverage remained the same at 97.778% when pulling 76d0247 on feugy:master into c624cbb on power-assert-js:master.

@twada
Copy link
Member

twada commented Dec 25, 2017

@feugy Ah nice catch. Thanks!

@twada twada merged commit 7206df0 into power-assert-js:master Dec 25, 2017
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

3 participants