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

feat: added --human-time flag #92

Closed
wants to merge 2 commits into from
Closed

feat: added --human-time flag #92

wants to merge 2 commits into from

Conversation

rachancheet
Copy link

--human-time flag uses more convenient 24-hour format rather than UNIX_EPOCH seconds when saving image to disk.

@rachancheet
Copy link
Author

if you want date as well , Please let me know .

@Shinyzenith
Copy link
Member

Thanks for the pr but I am mostly hesitant about adding new crates to the dependency graph. Is there some other options we can explore / features to disable other parts of introduced crate ( I don't have experience with the crate you are using and hence no knowledge of it either )

I like the feature however and the flag name could probably be better named for being intuitive. Also this needs to be documented if we wish to merge this change.

@Shinyzenith
Copy link
Member

Can you rebase this on feature freeze andreas branch?

@rachancheet rachancheet closed this by deleting the head repository Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants