Skip to content
This repository has been archived by the owner on Apr 22, 2020. It is now read-only.

Misc updates 0806 #189

Merged
merged 3 commits into from
Aug 19, 2019
Merged

Misc updates 0806 #189

merged 3 commits into from
Aug 19, 2019

Conversation

koreno
Copy link
Contributor

@koreno koreno commented Aug 6, 2019

No description provided.

easypy/colors.py Show resolved Hide resolved
@koreno
Copy link
Contributor Author

koreno commented Aug 13, 2019

@idanarye ?

@idanarye
Copy link

Still think the change to DataSize.__str__ is wrong.

…MiB'

this improves compatibility with 3rd party tools that dislike the space inside the value, and is more consistent with higher values ('5bytes', '5MiB')
backwards compatibilty is retained in the form of a special formatting specifier: `"{:text}".format(byte) == "1 byte"`
@koreno
Copy link
Contributor Author

koreno commented Aug 19, 2019

Still think the change to DataSize.__str__ is wrong.

I've compensated with a new .render and .__format__ implementation

@koreno koreno merged commit accd639 into master Aug 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants