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

Issue 42 ns fails #194

Closed
wants to merge 2 commits into from
Closed

Issue 42 ns fails #194

wants to merge 2 commits into from

Conversation

zbeekman
Copy link
Collaborator

I found a typo that I introduced in c7022fe which may be responsible for #42.

I have fixed the typo, and re-enabled the NS tests on Travis-CI in hopes that they now pass.

@codecov-io
Copy link

codecov-io commented May 31, 2016

Current coverage is 49.72%

Merging #194 into master will not change coverage

@@             master       #194   diff @@
==========================================
  Files             2          2          
  Lines           895        895          
  Methods          42         42          
  Messages          0          0          
  Branches        149        149          
==========================================
  Hits            445        445          
  Misses          450        450          
  Partials          0          0          

Sunburst

Powered by Codecov. Last updated by f3f6a78...3ccc280

@zbeekman zbeekman force-pushed the issue-42-NS-fails branch from 47d28bd to 6b47189 Compare June 4, 2016 18:11
@zbeekman
Copy link
Collaborator Author

zbeekman commented Jun 4, 2016

Once tests pass merge this, then merge #195

@zbeekman
Copy link
Collaborator Author

zbeekman commented Jun 4, 2016

Actually, much simpler to just merge #195 and close this PR.

@zbeekman zbeekman closed this Jun 4, 2016
@zbeekman zbeekman deleted the issue-42-NS-fails branch June 4, 2016 19:24
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