autocomplete icon indicating copy to clipboard operation
autocomplete copied to clipboard

feat: add `ykman` spec

Open vaaleyard opened this issue 3 years ago • 3 comments

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) feature, new spec for ykman What is the current behavior? (You can also link to an open issue here) Done

Additional info: This is the yubikey-manager CLI: https://github.com/Yubico/yubikey-manager

vaaleyard avatar Jul 24 '22 22:07 vaaleyard

Overview

src/ykman.ts:

Info:

Script: ykman list | sed -rn 's/.*Serial: (.*)/\1/p' postProcess(function):

 function (out) {
            return out.split("\n").map((serial) => {
              return { name: serial, description: "Yubikey serial" };
            });
          }

Script: ykman list --readers postProcess(function):

 function (out) {
            return out.split("\n").map((readerName) => {
              return { name: readerName, description: "Yubikey name" };
            });
          }

withfig-bot avatar Jul 24 '22 22:07 withfig-bot

Hello @vaaleyard, thank you very much for creating a Pull Request! Here is a small checklist to get this PR merged as quickly as possible:

  • [ ] Do all subcommands / options which take arguments include the args property (args: {})?
  • [ ] Are all options modular? E.g. -a -u -x instead of -aux
  • [ ] Have all other checks passed?

Please add a 👍 as a reaction to this comment to show that you read this.

withfig-bot avatar Jul 24 '22 22:07 withfig-bot

Fantastic, will be ready to merge once that simple change is made - I get a lot of notifications on github so please do bump this if I miss it :)

clo4 avatar Aug 02 '22 04:08 clo4

I guess it's all okay now?

vaaleyard avatar Sep 22 '22 20:09 vaaleyard