password icon indicating copy to clipboard operation
password copied to clipboard

Add documentation about the `password-cli` (#83)

Open blackheaven opened this issue 2 months ago • 6 comments

A bit of documentation, let me know if you had something else in mind.

blackheaven avatar Oct 04 '25 21:10 blackheaven

I don't think the CI failure is related to my PR.

blackheaven avatar Oct 04 '25 22:10 blackheaven

Ah, yes. optparse-applicative has a newer version we can probably allow.

Not sure if I like the repetition of the Testing segment in all the modules. But then again, it's good information close to the usage site of the library. I might have to think about it a bit more.

Vlix avatar Oct 07 '25 17:10 Vlix

Ah, nevermind. I've literally asked for that in the #83 issue. 🤦

I would like to also see the second part of the checkboxes of the issue's first post ticked in this PR: Mostly changing the "SOME-HASH" sections in the password-cli/README to 'SOME-HASH' and adding a comment that using " might interpolate in some cases (e.g. when a hash or password has a $ in it)

Vlix avatar Oct 07 '25 18:10 Vlix

Ah, nevermind. I've literally asked for that in the #83 issue. 🤦

I would like to also see the second part of the checkboxes of the issue's first post ticked in this PR: Mostly changing the "SOME-HASH" sections in the password-cli/README to 'SOME-HASH' and adding a comment that using " might interpolate in some cases (e.g. when a hash or password has a $ in it)

Done :)

blackheaven avatar Oct 07 '25 20:10 blackheaven

@Vlix can you have another look please?

blackheaven avatar Oct 08 '25 20:10 blackheaven

@Vlix thanks for you feedback, ready for a new review.

blackheaven avatar Oct 10 '25 20:10 blackheaven