-
Notifications
You must be signed in to change notification settings - Fork 15
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
Add pretty_num for linear quantities #26
Conversation
minimal documentation
improve lisibility fix doc typo
extand the prefixes to latest 1991 list
remove 'missing description' warning on documentation
Codecov Report
@@ Coverage Diff @@
## master #26 +/- ##
===========================================
+ Coverage 65.26% 76.25% +10.99%
===========================================
Files 4 5 +1
Lines 95 139 +44
===========================================
+ Hits 62 106 +44
Misses 33 33
Continue to review full report at Codecov.
|
hello @gaborcsardi, is there something I should change on that P.R. ? |
@cregouby I am sorry for the delay, I'll try to look at this this weekend. 🤞 |
We can't use non-ascii characters unconditionally, unfortunately.
Need to update nrow if we rule out some prefixes.
We need a tolerance here.
Thank you! |
...as a fix to #8