Add documentation about the password-cli (#83)#86
Conversation
|
I don't think the CI failure is related to my PR. |
|
Ah, yes. Not sure if I like the repetition of the |
|
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: |
5f7bdee to
2daa749
Compare
Done :) |
|
@Vlix can you have another look please? |
|
|
||
| You can use [password-cli](https://github.com/cdepillabout/password/tree/master/password-cli) to test it: | ||
|
|
||
| > $ password-cli check argon2 --hash "SOME-HASH" |
There was a problem hiding this comment.
These kind of examples might also be better with single quotes '.
There was a problem hiding this comment.
oops, I have fixed it and the other ones
2daa749 to
00ea6cc
Compare
|
@Vlix thanks for you feedback, ready for a new review. |
Vlix
left a comment
There was a problem hiding this comment.
I think this is ready to be merged :)
I'll hold off on actually merging until publishing the password-cli to hackage. Don't want to reference it in documentation when it's not actually available.
A bit of documentation, let me know if you had something else in mind.