Skip to content

add total duration in 'hamster list' command#380

Merged
ederag merged 3 commits intoprojecthamster:masterfrom
madtibo:total_duration_in_list
Feb 3, 2019
Merged

add total duration in 'hamster list' command#380
ederag merged 3 commits intoprojecthamster:masterfrom
madtibo:total_duration_in_list

Conversation

@madtibo
Copy link
Contributor

@madtibo madtibo commented Jan 22, 2019

Add a total time line at the bottom of the 'hamster list' command.

@ederag
Copy link
Collaborator

ederag commented Jan 23, 2019

Thanks for your contribution.
It would be better to just sum deltas (that are timedelta),
and use stuff.format_duration for the display ?

@madtibo
Copy link
Contributor Author

madtibo commented Jan 25, 2019

I saved the duration in seconds, is it better that way?

@ederag
Copy link
Collaborator

ederag commented Feb 2, 2019

What about 4397d26 ?
Totals would use the same "human" format as in the duration column.

@madtibo
Copy link
Contributor Author

madtibo commented Feb 3, 2019

Perfect!

@ederag ederag merged commit 79db77f into projecthamster:master Feb 3, 2019
@ederag
Copy link
Collaborator

ederag commented Feb 3, 2019

Thanks for contributing !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants