sleek icon indicating copy to clipboard operation
sleek copied to clipboard

Unable to use `--uppercase=false` (or any other boolean options)

Open mitranim opened this issue 11 months ago • 1 comments

The readme states that --uppercase is boolean and true by default. When providing --uppercase=false, I get the following error:

error: unexpected value 'false' for '--uppercase' found; no more were expected

Get similar errors for other boolean options. They all seem broken.

mitranim avatar Feb 07 '25 14:02 mitranim

By the way, the commit aa29cb8c283f810acbe47851d1059e56e72bdf7e needlessly complicates the option. It seems cleaner to revert it to bool and supply default_value = "true" (tested, works when no CLI argument is provided). Although I think lowercase is the better default.

mitranim avatar Feb 07 '25 14:02 mitranim

Thank you for the report!

I'm working on a massive overhaul now that should fix this.

https://github.com/nrempel/sleek/commit/dc5293a1e74eed59dd6e05283bce9862e0dbdd4b

nrempel avatar May 24 '25 15:05 nrempel