Skip to content

Improved Coverage#41

Open
powerbrian wants to merge 1 commit into
masterfrom
Improving-Coverage
Open

Improved Coverage#41
powerbrian wants to merge 1 commit into
masterfrom
Improving-Coverage

Conversation

@powerbrian
Copy link
Copy Markdown
Contributor

Up to 89.29%

Up to 89.29%
@mcdonnelldean
Copy link
Copy Markdown
Contributor

@powerbrian to get this building please see nodezoo-npm. You need to update the following.

  • scripts and dependencies in package.json
  • travis file
  • gitignore

I PR'd a load of stuff that fixes both travis and coveralls, if you also add it here everything will go green.

@powerbrian
Copy link
Copy Markdown
Contributor Author

@mcdonnelldean Was this broken before I added the extra tests? Seems to be giving out about lack of .travis.yml file. It's a good exercise for me to look at. It is not building locally for me. If it was working I may revert and add the tests again, if not I will try and sort it.

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.

2 participants