Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #26 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 2 2
Lines 56 57 +1
=========================================
+ Hits 56 57 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
sostock
left a comment
There was a problem hiding this comment.
Thanks for your contribution. Since the convenient shortcut u(e) instead of uconvert(u, e) is available (as used in the original issue), I think it could be mentioned in the docstring and should be tested as well. That’s why I propose the following changes.
Sounds good! Committing |
Use units themselves as callables to do `uconvert` Co-authored-by: Sebastian Stock <42280794+sostock@users.noreply.github.com>
Solves #21