-
Notifications
You must be signed in to change notification settings - Fork 37
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
Status sub command #91
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Oskar Kohout <oskar@evilbit.at>
Signed-off-by: Oskar Kohout <oskar@evilbit.at>
Signed-off-by: Oskar Kohout <oskar@evilbit.at>
Signed-off-by: Oskar Kohout <oskar@evilbit.at>
Signed-off-by: Oskar Kohout <oskar@evilbit.at>
Signed-off-by: Oskar Kohout <oskar@evilbit.at>
ef209fa
to
708e250
Compare
Codecov Report
@@ Coverage Diff @@
## master #91 +/- ##
==========================================
- Coverage 81.75% 75.54% -6.21%
==========================================
Files 4 4
Lines 822 822
==========================================
- Hits 672 621 -51
- Misses 150 201 +51
Continue to review full report at Codecov.
|
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.
7f4f4bd
to
979ba11
Compare
The whitespace checker fail is anticipated - the two trailing whitespaces are necessary to coerce pandoc into rendering sensible man pages. |
**tpm2-totp -P** *-* **init** | ||
\> *verysecret***\<CTRL-D\>** |
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.
The whitespace checker fail is anticipated - the two trailing whitespaces are necessary to coerce pandoc into rendering sensible man pages.
doesn't cut it here.
I'd prefer to use a backslash, followed directly like a line break instead of the trailing whitespace to be able to keep the whitespace checker without modification, otherwise LGTM!
**tpm2-totp -P** *-* **init**\
\> *verysecret***\<CTRL-D\>**
\# or (recommended) | ||
**gpg --decrypt** */path/to/password.gpg* **|** **tpm2-totp** **-P** *-* **init** |
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.
Same as above.
\# or (discouraged) | ||
**tpm2-totp -P** *verysecret* **init** |
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.
Same as above.
implements #85
Generates valid YAML as output, so can easily be processed with e.g.
yq
while still remaining reasonably appealing for human consumption.Tip: try piping into
bat -l yaml -pp
for CLI syntax highlighting.Example:
tpm2-totp status
:Some
keyBlob
handling code has been refactored to reduce repetition.Also,
man
page formatting is much nicer now (fixed wrong formatting forpandoc
toman
).