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

update to using a newer version of Babel and nyc for coverage #10

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bcoe
Copy link
Owner

@bcoe bcoe commented Dec 1, 2015

This fixes coverage by removing blanket and pulling in nyc + istanbul, I've also updated babel using its new approach for instrumentation.

  • update to a newer version of babel.
  • pull in a newer version of nyc for instrumenting test coverage.

@ceejbot we'll definitely want to test this on a host before we release it to production -- might be worth making this a major version bump.

I've added you as an owner to the npm module and to the GitHub repo.

@ceejbot
Copy link
Collaborator

ceejbot commented Dec 1, 2015

I've been using my fork locally just to get the config dump feature because I needed it. Will pull this branch & test.

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

2 participants