refactor: convert locations:modes:(create|delete) commands to yargs#738
Conversation
|
| const builder = (yargs: Argv): Argv<CommandArgs> => | ||
| apiCommandBuilder(yargs) | ||
| .positional('id', { describe: 'mode id', type: 'string' }) | ||
| .option('location', { alias: 'l', describe: 'a specific location to create the mode in', type: 'string' }) |
There was a problem hiding this comment.
"to delete the mode from"
There was a problem hiding this comment.
Oops. Ugh, this is another one of those that's super weird to write correctly. It's not really "to delete the mode from" either though, it's just a filter for when the user doesn't specify a node. I've changed it to the generic "to query" we've been using everywhere else.
| 'delete the mode with the specified id', | ||
| ], | ||
| ]) | ||
| .epilog(apiDocsURL('createMode')) |
There was a problem hiding this comment.
Something along the lines of deleteMode?
rossiam
force-pushed
the
yargs-locations-modes-create-and-delete
branch
from
44fd2e0 to
710c8b0
Compare
rossiam
force-pushed
the
yargs-locations-modes-create-and-delete
branch
from
710c8b0 to
a72a54f
Compare
2 participants
locations:modes:createandlocations:modes:deletecommands to yargs