Skip to content
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

Clean up all these Time.at calls #22

Closed
glacials opened this issue Feb 1, 2014 · 0 comments
Closed

Clean up all these Time.at calls #22

glacials opened this issue Feb 1, 2014 · 0 comments
Labels
innards-only Issues invisible to users

Comments

@glacials
Copy link
Owner

glacials commented Feb 1, 2014

Basically everywhere that uses human-friendly time formats (e.g. HH:MM:SS.NN), which includes the show view and some of the downloadable formats, needs to call something resembling Time.at(some_duration_in_seconds).utc.strftime("%k:%M:%S.%N").strip, which makes for a lot of (sometimes minorly different) identical calls.

We should abstract this to something that we can do things like split.duration.minutes or split.duration.seconds on, or maybe even split.duration.pretty if it turns out that all of the formats expect the same style string.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
innards-only Issues invisible to users
Projects
None yet
Development

No branches or pull requests

1 participant