dthree/vorpal

.catch should not trigger on a command group

Open

#64 ouverte le 21 déc. 2015

Voir sur GitHub
 (2 commentaires) (1 réaction) (0 assignés)JavaScript (5 609 stars) (313 forks)batch import
bughelp wanted

Description

when not implementing a catch hook, typing in a command of a group gives you the help for it's subcommands.

if however you implement a catch hook, the catch hook is triggered instead.

I think it would still be useful to show the help for a command group, instead of triggering the catch.

Guide contributeur

Stack technique
javascriptnodejs
Domaine
cli
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Node.jsbasic Vorpal knowledge
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
Examine Vorpal's command group and catch hook logic in the source code (likely in lib/command.js). Understand how the library distinguishes between a command group and a regular command. The current behavior triggers catch when a group is invoked; the expected behavior is to show help for the group. Modify the command execution flow to skip catch for command groups and instead display help.