-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[results.webkit.org] Display elapsed hours for long runs #14361
[results.webkit.org] Display elapsed hours for long runs #14361
Conversation
EWS run on current version of this PR (hash b367e72) |
if (time.getMinutes()) | ||
result += `${time.getMinutes()} minute${time.getMinutes() == 1 ? '' : 's'} and `; | ||
result += `${time.getSeconds()} second${time.getSeconds() == 1 ? '' : 's'} to run`; | ||
const elapsed = Math.round(endTimestamp - startTimestamp); |
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.
can make it a separate method as well, e.g.: convertSecondsToReadableDuration
https://bugs.webkit.org/show_bug.cgi?id=257344 rdar://109849743 Reviewed by Aakash Jain. * Tools/Scripts/libraries/resultsdbpy/resultsdbpy/view/static/js/common.js: Convert elapsed seconds to a human-readable time without using the Date class. Canonical link: https://commits.webkit.org/264589@main
b367e72
to
413f417
Compare
Committed 264589@main (413f417): https://commits.webkit.org/264589@main Reviewed commits have been landed. Closing PR #14361 and removing active labels. |
413f417
b367e72
π§ͺ ios-wk2π§ͺ api-macπ§ͺ gtk-wk2π§ͺ api-gtkπ§ͺ mac-AS-debug-wk2