Use µs not us #42
Open
Use µs not us #42
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
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! ;-) |
Open
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.