Possibility to save one character in width to make it 80 columns

[CouldBeMathijs]

Hi!

I realize that this project does not normally accept feature requests. But the only tiny gripe I have with it is that the output is 81 characters in width while most terminal emulators open with 80 columns. This makes it so the last closing bracket of the Storage value wraps around and breaks up the nix logo. Would you consider removing one white space from either side of the nix logo, or make the storage option one digit less precise?

If not that's fine too of course, I'll just have to deal

[NotAShelf]

I realize that this project does not normally accept feature requests.

Fwiw I'm generally open to feature requests, it's just that there aren't much features that are in scope OR can be implemented without incurring a performance penalty. Either way, I'm sorry that I gave you this impression; feature requests are welcome :)

But the only tiny gripe I have with it is that the output is 81 characters in width while most terminal emulators open with 80 columns. This makes it so the last closing bracket of the Storage value wraps around and breaks up the nix logo. Would you consider removing one white space from either side of the nix logo, or make the storage option one digit less precise?

Sure! I never noticed an issue with the width, or that the widh is exactly 80 columns, but I'll implement a hotfix now. This project has been stale complete for a while but another release wouldn't hurt.

[NotAShelf]

Should be resolved in 1d69d31

[CouldBeMathijs]

Oops, that's my fault, I misunderstood the README.
Thanks for the quick fix, I look forward to seeing it enter my nix-unstable channel in the coming weeks.

[NotAShelf]

It's already tagged, so you can expect an automated PR sometime this week probably. I'm working on a small PR (#22) to improve performance, and 0.4.11 will be available shortly after I'm done. If the PR is not made by then, I'll probably create a PR myself to bump it in nixpkgs.

FWIW you acn also use the flake for the time being, but I guess that's a bit too much effort just for a fetch "script".

[CouldBeMathijs]

Hi! Any update on this? Because it seems like the automated PR has not been made yet, and both master and unstable still ship 0.4.9 as of today

[NotAShelf]

I'm going to make another release soon, which is why I have been holding off on the manual PR for the time being. Though I can probably merge #29 today and make the release after fixing my own review comment. However I'm a little preoccupied due to real life stuff, so I'd encourage you to use thte flake for now; since I've stripped most of the deps, compilation is pretty fast.