-
Notifications
You must be signed in to change notification settings - Fork 72
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
CLI changes needed for BBS+ #334
base: main
Are you sure you want to change the base?
Conversation
…ith BBS+ derived credentials
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.
Looks good, sorry for the messiness of the CLI, we need to refactor it.
I'll review the ssi
PR another day
@@ -29,6 +29,9 @@ use didkit_cli::opts::ResolverOptions; | |||
|
|||
#[derive(StructOpt, Debug)] | |||
pub enum DIDKit { | |||
#[clap(setting(clap::AppSettings::Hidden))] | |||
GenerateBls12381Key, |
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.
Could you put it in KeyGenerateCmd
? Then the command will be didkit key generate bls12381
@@ -999,6 +1051,7 @@ fn main() -> AResult<()> { | |||
false, | |||
Some(&options), | |||
&mut loader, | |||
false |
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 can rebase on main
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.
just blocking to avoid any premature merge
No description provided.