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

Make SI and IEC standards defaults #138

Merged
merged 1 commit into from
Aug 10, 2021

Conversation

getsnoopy
Copy link
Contributor

This commit makes the SI standard (base 10) the generic default, and the
IEC standard the default for base-2 formatting in order to make it as
intuitive as possible, to reduce any ambiguity, and to mirror the
industry standard.

Fixes #137

This commit makes the SI standard (base 10) the generic default, and the
IEC standard the default for base-2 formatting in order to make it as
intuitive as possible, to reduce any ambiguity, and to mirror the
industry standard.

Fixes avoidwork#137
@avoidwork
Copy link
Owner

@getsnoopy I don't want to make SI default; just IEC if you feel it needs to change. There's millions of projects using this directly and indirectly, and base 2 to 10 would impact them.

@avoidwork avoidwork merged commit b2ec833 into avoidwork:master Aug 10, 2021
@avoidwork
Copy link
Owner

merged it anyways, 'cause this should happen (aka 2.0.0).

@getsnoopy
Copy link
Contributor Author

This would be a major version bump, so no one currently using this would be affected by it; only new users.

@getsnoopy getsnoopy deleted the make-si-default branch August 10, 2021 16:49
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.

SI standard should be default
2 participants