Skip to content

Conversation

@MyUsernamee
Copy link

This pull request adds the [TIME_FORMAT] options to the serve sub command that allows for different formats of time returned from the script.

It takes an optional argument, which can be either m, s, or m:s which formats the time as, minutes (same as original), seconds, or a string "MM:SS".

This has no effect on the original behavior unless that option is specified.

I know this changes was mentioned in #3, where it was said that this kind of feature wasn't planned, and as such I understand if this PR is unwelcome, and I apologize.

If there is anything I need to change let me know, and thank you for your time, and an amazing project! :)

Squashed:

First changes, Add TimeFormat Enum, impls, and basic logic

Fix formatting bug with seconds

Fix empty string with no timer

Restructure format minutes and seconds, fix bug

Fix zero second bug
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.

1 participant