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
Allow an optional separator splitting the value and unit of the result of ByteSize.human_readable
.
#8706
Conversation
CodSpeed Performance ReportMerging #8706 will not alter performanceComparing Summary
|
Please review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it'd make more sense to support a str
type separator, which is an empty string by default. Could you please use that approach instead?
Thanks for your work here. Looking forward to having this new feature.
OK. Changes committed. |
ByteSize.human_readable
by a space character.ByteSize.human_readable
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks! So excited to have this as a new feature in our upcoming minor release :).
Thanks so much! It's my pleasure. |
Change Summary
Allow splitting the value and unit of the result of
ByteSize.human_readable
with a separator providing by the argumentseparator
.Related issue number
fix #8668
Checklist
Selected Reviewer: @adriangb