-
Notifications
You must be signed in to change notification settings - Fork 49
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
Type preservation #164
Type preservation #164
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks so much for your contribution @znicholls! And sorry for letting this go stale.
Please could you rebase, and service my suggestion for making the code more generic?
d67e9a7
to
c1d0279
Compare
Codecov Report
@@ Coverage Diff @@
## master #164 +/- ##
==========================================
+ Coverage 90.48% 90.61% +0.13%
==========================================
Files 7 7
Lines 830 831 +1
Branches 105 105
==========================================
+ Hits 751 753 +2
Misses 64 64
+ Partials 15 14 -1
Continue to review full report at Codecov.
|
No worries, lots happening
Done |
Closes #163