search

tailhook / humantime

A parser and formatter for std::time::{SystemTime, Duration}
Apache License 2.0
283 stars 34 forks source link

Use µs not us #42

Open RobertGauld opened 9 months ago

RobertGauld commented 9 months ago

At the risk of starting a debate µ (not u) is the correct SI prefix for micro.

I've deliberatly split the PR into 2 commits incase you want to support parsing but not generating the correct micro prefix.

ptdecker commented 9 months ago

If this is added, can it be done as a feature switch? I have a very significant corporate customer (a large global financial institution) who has dinged us (a small start up) for delivering logs that contain µ. I'm using this in a CLI application because it is so awesome but if it starts outputting the unicode symbol again they will freak out. Thank you for considering the welfare of your fellow coders! ;-)

RobertGauld commented 9 months ago

Thanks @ptdecker, that suggestion's in #45