dthree/vorpal

.catch should not trigger on a command group

Open

#64 geöffnet am 21. Dez. 2015

Auf GitHub ansehen
 (2 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (5.609 Stars) (313 Forks)batch import
bughelp wanted

Beschreibung

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.

Contributor Guide

Tech Stack
javascriptnodejs
Domain
cli
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Node.jsbasic Vorpal knowledge
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60
Research-Richtung
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.