Add help for $ phenomic and unknown commands
#757
Conversation
| .help() | ||
| .showHelpOnFail() |
There was a problem hiding this comment.
It's not working at all since we don't any validations with yargs.check(). But this is actually an interesting idea. I could move the validation part to yarts.check() and let yargs its jobs.
|
I'm busy right now. Will do this tonight |
|
Yeah sure, no rush :) |
|
I updated the PR. Some screenshots:
|
| @@ -6,7 +6,8 @@ import definitions from "./definitions.js" | |||
|
|
|||
| yargs | |||
| .version(() => version) | |||
| .help() | |||
| .usage("Usage: phenomic <command> [options]") | |||
| // .hep() | |||
| const currentCommand = argv._[2] | ||
| if ([ "start", "build", "setup" ].indexOf(currentCommand) < 0) { | ||
| throw new Error( | ||
| colors.bgRed(colors.white("ERROR")) + |
There was a problem hiding this comment.
We should probably try to output a more fancy error prefix than "ERROR" :D
There was a problem hiding this comment.
@MoOx what are you suggesting?
| // nodePath, | ||
| // pathToPhenomicScript, | ||
| // input argvs | ||
| // ] |
There was a problem hiding this comment.
Can you remove this commented code?
There was a problem hiding this comment.
OK. I added this on purpose to explain about argv._.length < 3
|
I'll add some tests with |
|
Also add |
|
Will be handled in #936 |
Handle some small details of CLI:
$ phenomicwithout any command/ options (Closes$ phenomicdoes nothing #513)