Exclude option ignored unless in nyc stanza of package.json

[wKovacs64]

When using babel-plugin-istanbul, it appears if the nyc configuration object is included in the nyc stanza of package.json, everything is fine. But when attempting to use the same options in .nycrc or via the command-line, they are ignored (exclude, in particular).

Repro examples: https://github.com/wKovacs64/nyc-exclude-test (deleted)

Side note: in my real projects where I first ran into this, I require babel-register with mocha as opposed to nyc. The nyc documentation does it in the nyc configuration, so that's what I used in my reproduction cases, but it doesn't appear to make a difference.

[tteltrab]

I just recreated this as well. The exclude option in a .nycrc file when using this package is ignored, but the files are ignored when the nyc configuration is in the package.json.

[bcoe]

@tteltrab @wKovacs64 I believe there's another issue open on nyc related to this; will dig soon, any help appreciated.

[321ckatz123]

@bcoe the issue is here: istanbuljs/nyc#419.

[bcoe]

@321ckatz123 @wKovacs64 I believe we've landed a patch for this, could you give the following a try:

npm cache clear; npm i babel-plugin-istanbul@next

Thanks to @alpersogukpinar for identifying the underlying problem.

[wKovacs64]

@bcoe babel-plugin-istanbul@next (4.0.0) is working as expected, my exclusion tests pass for command-line, .nycrc, and package.json. Thanks to all involved!

[321ckatz123]

@bcoe me as well. When can we get this pushed to npm?

[bcoe]

@321ckatz123 the module is up, just not on latest yet; try this:

npm cache clear; npm i babel-plugin-istanbul@next.

[321ckatz123]

@bcoe, I understand how to get it now. I was more asking when is the next release (which I assume includes this) is going to get pushed?

[bcoe]

@wKovacs64 @321ckatz123 this is published to latest now 👍