Skip to content

Conversation

@ddelange
Copy link
Collaborator

Closes #16

@ddelange ddelange requested a review from cpcloud December 15, 2020 10:59
@ddelange ddelange force-pushed the timestamp branch 4 times, most recently from 41c4513 to 0d47f35 Compare December 15, 2020 14:50
@cpcloud
Copy link
Owner

cpcloud commented Dec 15, 2020

Is there any chance this could be configurable, so that users can still get the original behavior without the started timestamp?

@ddelange
Copy link
Collaborator Author

ddelange commented Dec 15, 2020

Not neatly I'm afraid.. ref #18 (comment)

Do you think users depend on it the way it is now? I'm also not 100% confident about this (breaking?) change, but I concluded it can't do any harm. Please prove me wrong though!

Copy link
Owner

@cpcloud cpcloud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Merging. Thank you.

@cpcloud cpcloud merged commit 38ae41a into cpcloud:master Dec 19, 2020
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.

Feature request: also print timestamp

2 participants