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

Reach 90% test coverage #13

Open
tsalo opened this issue Aug 12, 2018 · 1 comment
Open

Reach 90% test coverage #13

tsalo opened this issue Aug 12, 2018 · 1 comment

Comments

@tsalo
Copy link
Contributor

tsalo commented Aug 12, 2018

Per #12, the goal is to reach 90% coverage for the 2.0.0 release. @bbfrederick would you be willing to create a milestone for 2.0.0 and assign this issue to that milestone?

@bbfrederick
Copy link
Owner

After a burst of activity a few weeks ago, I managed to get coverage up to 46% (from languishing at 19 where it had for quite some time). Getting to this point actually helped me find and exterminate several bugs. I'm starting to thing that 90% coverage may be too optimistic though. I can certainly see getting higher than were we are. I'll put some more work into this in the next few weeks and see where we get to. I've been building up a number of compatibility breaking interface changes, and would like to switch over to the new option structure, but that should be done in a major number release...

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

No branches or pull requests

2 participants