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
[DSCP-351] improve keygen commands #305
Conversation
b06995b
to
b9256a5
Compare
tools/src/keygen/KeygenOptions.hs
Outdated
@@ -32,7 +31,7 @@ secretDataTypeParser = asum | |||
, CommSecret <$> option auto | |||
(long "comm-sec" <> | |||
metavar "N" <> | |||
help "Use given committee secret and derive secret key of N-th \ | |||
help "Use giencodeIntegerven committee secret and derive secret key of N-th \ |
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.
You've mistakenly inserted encodeInteger
in the middle of the other word.
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.
:wtf:
tools/src/keygen/KeygenOptions.hs
Outdated
|
||
, command "student-submission" $ info | ||
(fmap PrintStudentSubmission . strOption $ | ||
long "seed" <> |
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.
Is there any other way to solve this?
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.
I extracted that parser and one below, if you were talking about it.
6ef1591
to
bec5a87
Compare
bec5a87
to
b5411d8
Compare
Description
Improves keygen interface. Adds help messages for each command of the keygen.
For instance:
dscp-keygen --seed --command "address:hex"
->dscp-keygen --seed address --hex
.YT issue
https://issues.serokell.io/issue/DSCP-351
Checklist
Hint: a perfect PR has all the checkmarks set.
Possible related changes (conditional):
Stylistic (obligatory):