-
Notifications
You must be signed in to change notification settings - Fork 73
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
Tests, code coverage and some new functionality #43
Conversation
Codecov Report
@@ Coverage Diff @@
## master #43 +/- ##
=========================================
Coverage ? 87.78%
=========================================
Files ? 6
Lines ? 794
Branches ? 0
=========================================
Hits ? 697
Misses ? 97
Partials ? 0
Continue to review full report at Codecov.
|
I added a badge. Great set of tests - thanks! |
Hopefully if we split out some of the functionality the tests can become a bit more granular, but it's a good start. I'm glad to have put them in, because they keep catching mistakes :) |
This implements native plotly heatmaps (#20), font size params (#39), cellnote (#42), support for long data and altered mouseover labels (#37 and #36).
I will double check these edits before merging, this is merely an easy way to compare.
Basic tests and code coverage reporting are implemented here. I'm not 100% about how to add a coverage badge, though you could make an account on
codecov.io
to add it yourself