-
Notifications
You must be signed in to change notification settings - Fork 266
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 decimal place support #125
base: main
Are you sure you want to change the base?
Conversation
ping |
* @return {String} | ||
* @api private | ||
*/ | ||
|
||
function fmtLong(ms) { | ||
function fmtLong(ms, demical) { |
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.
I think this should be decimal
, right?
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.
uhm yeah
Is there literally any chance of this being implemented? I use it for years now, works flawlessly but it's a bit of hassle to always remember and look for the correct fork every time. It shouldn't even have any breaking features, only adding a one new (and quite useful one imho). |
As per #124
Here is the PR to add decimal place support. Default is
0
, max decimal place is3
.