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

Make nyc instrument work in subdirectories #343

Merged
merged 1 commit into from
Aug 9, 2016

Conversation

addaleax
Copy link
Member

@addaleax addaleax commented Aug 9, 2016

Hi everyone! Ran into a problem running nyc instrument today, and it would be cool if you could take a look at. I’m pretty sure this is a correct fix, but still, removing code always comes with the suspicion that there is a reason it was there in the first place. ;)


There is no real reason to use a relative path when performing the I/O for instrumenting. Absolute paths should work just as well and will be correct, even when nyc.cwd is off a bit because it is modified based on what pkgUp says.

There is no real reason to use a relative path when performing the
I/O for instrumenting. Absolute paths should work just as well
and will be correct, even when `nyc.cwd` is off a bit because it
is modified based on what `pkgUp` says.
@coveralls
Copy link

coveralls commented Aug 9, 2016

Coverage Status

Changes Unknown when pulling 86a1b54 on addaleax:fix-instrument-in-subdir into * on istanbuljs:master*.

@bcoe
Copy link
Member

bcoe commented Aug 9, 2016

@addaleax this patch looks quite reasonable to me; I think this path issue was probably more related to me always getting tripped up by the paradox of choice in the fs API, rather than a specific issue I was trying to solve.

@addaleax
Copy link
Member Author

addaleax commented Aug 9, 2016

@bcoe Ok, cool then! Everything’s green, I guess there’s no reason not to merge this?

@bcoe
Copy link
Member

bcoe commented Aug 9, 2016

@addaleax 👍 might take me a couple days to get a release out, I want to carefully land a breaking test-exclude API change.

@bcoe bcoe merged commit a82cf49 into istanbuljs:master Aug 9, 2016
@addaleax addaleax deleted the fix-instrument-in-subdir branch August 9, 2016 17:48
@addaleax
Copy link
Member Author

addaleax commented Aug 9, 2016

Sure, I guess I’ll just keep an eye on nyc@next this week? :)

@bcoe
Copy link
Member

bcoe commented Aug 12, 2016

@addaleax give nyc@next a spin, I've released your changes; I'd also love the extra set of eyes on the exclude/include logic, as I've introduced a breaking change.

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.

3 participants