Resolve "Keys clean subcommand : Indicate to Structopt that the user must indicate which keypair to delete"

J'ai utilisé clap::arg_enum! indirectement en dessous de ClearOpt ce qui fait une struct opt avec un seul attribut enum. Il n'est pas possible actuellement que ce soit ClearOpt qui soit l'enum puisque il y a un ticket mergé mais non sorti dans une version clap qui bloque les commentaires de doc sur les entrées de l'enum dans arg_enum et donc fait échouer le check sur la doc des variants des enum StructOpt.

added S-request review label

@dvermd merci c'est très bien mais tu a oublié un besoin : demander a l'utilisateur confirmation qu'il veut bien supprimer tel clé. Tu peut t'inspirer de la commande wizard pour voir comment interagir avec l'utilisateur :)

added S-waiting author label and removed S-request review label

added 1 commit

1dc4dd92 - [fix] core:commands: #108 (closed) rework keys cli options

Compare with previous version

added 2 commits

8ad244ae - 1 commit from branch dev
3f30cd7c - [fix] core:commands: #108 (closed) rework keys cli options

Compare with previous version

added 1 commit

d385f7f7 - [fix] core:commands: #108 (closed) rework keys cli options

Compare with previous version

@dvermd quand ton code est pret a merger pense a me prévenir et me taggeant ici. Pense également a remplacer le label S-awaiting author par le label S-request review, merci :)

added S-request review label and removed S-waiting author label

@librelois normalement c'est bon. J'ai été obligé de créer une fonction privée pour garder les tests et demander la confirmation utilisateur dans la première fonction

@dvermd ok ça m'arrange, je suis en train de faire une grosse refacto de la crate conf et ça rentre en conflit avec ton code, il vaut mieux que je merge le tiens maintenant pour que ce soit moi qui gère les conflits :)

Concernant ton code c'est très bon, je vois que tu a bien progressé, je n'ai pas de retour bloquant, je vais qualifier la feature (test manuel) puis merger :)

Resolve "Keys clean subcommand : Indicate to Structopt that the user must indicate which keypair to delete"

Merge request reports

Activity