-
-
Notifications
You must be signed in to change notification settings - Fork 360
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 check-coverage with custom options #483
Conversation
Specifically, if the temp directory is configured, this needs to be configured on the NYC instance for both the command line action `check-coverage` as well as the `--check-coverage` flag passed when executing tests.
Changes Unknown when pulling 40605d4 on wbyoung:fix-check-coverage into ** on istanbuljs:master**. |
@wbyoung looks quite reasonable to me 👍 thanks for the contribution. |
@wbyoung thanks for the contribution, hopefully the first of many; I'm hoping to have at least a candidate release out to the |
@bcoe awesome — thanks for merging & releasing quickly! |
@wbyoung would love your help testing both of your patches:
once we're happy with our smoke testing, I'll promote this to |
@bcoe it looks like you released already… I have two projects that are now failing after this release. See wbyoung/relateify#1 and wbyoung/babel-plugin-transform-postcss#3. The output I get is:
|
try |
Specifically, if the temp directory is configured, this needs to be configured on the NYC instance for both the command line action
check-coverage
as well as the--check-coverage
flag passed when executing tests.