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

Change global Istanbul use to local #80

Merged
merged 1 commit into from
Feb 27, 2017

Conversation

oliverjam
Copy link
Contributor

@oliverjam oliverjam commented Feb 27, 2017

Running istanbul cover test.js directly will not work as we only install it locally. This change should ensure the command works as expected.
Relates #79

quote> Running  directly will not work as we only install it locally. This change should ensure the command works as expected.
quote> Relates dwyl#79
@codecov-io
Copy link

codecov-io commented Feb 27, 2017

Codecov Report

Merging #80 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #80   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           2      2           
  Lines          33     33           
=====================================
  Hits           33     33

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3277367...9806a25. Read the comment docs.

Copy link
Member

@nelsonic nelsonic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@oliverjam thanks for fixing this. (we always prefer relative/local module usage)

@nelsonic nelsonic merged commit e5f1a4e into dwyl:master Feb 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants