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

Switch istanbul CLI to more recent nyc one #850

Merged
merged 1 commit into from
Dec 29, 2016
Merged

Conversation

astorije
Copy link
Member

nyc is made by the Istanbul people and is meant to integrate nicely with Babel.

Note that at the moment this is just a drop-in replacement of what we currently have in order to fix broken npm run coverage, therefore it does not support our recent ES6 stuff. Same lcov report gets generated in same directory than before.

Only visual changes is a more detailed output when running npm run coverage.

`nyc` is made by the Istanbul people and is meant to integrate nicely with Babel.
Note that at the moment this is just a drop-in replacement of what we currently have in order to fix broken `npm run coverage`, therefore it does not support our recent ES6 stuff.
@astorije astorije added the Type: Bug Issues that report and PRs that solve any defects that cause unexpected behaviors. label Dec 29, 2016
@astorije astorije requested a review from xPaw December 29, 2016 07:53
@xPaw xPaw merged commit 3240fad into master Dec 29, 2016
@xPaw xPaw deleted the astorije/fix-coverage branch December 29, 2016 10:21
@xPaw xPaw added this to the 2.2.0 milestone Dec 29, 2016
matburnham pushed a commit to matburnham/lounge that referenced this pull request Sep 6, 2017
Switch istanbul CLI to more recent nyc one
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Issues that report and PRs that solve any defects that cause unexpected behaviors.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants