istanbuljs/nyc

nyc check-coverage: new command flag

Open

#432 aperta il 1 nov 2016

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)JavaScript (392 fork)batch import
Great First Contributionenhancementhelp wanted

Metriche repository

Star
 (5468 star)
Metriche merge PR
 (Merge medio 1176g 10h) (2 PR mergiate in 30 g)

Descrizione

Hello! Just want to suggest new CLI flag for the check-coverage command executes it like all other flags are set to 100.

npm check-coverage --max

instead

nyc check-coverage --lines 100 --branches 100 --statements 100 --functions 100

It looks visually better and shorter in the package.json.

Also why not to be more configurable like --max 90 to sets everything to 100, but branches to 90.

Better flag name?

Guida contributor

Direzione di ricerca
Esplora la codebase di nyc, in particolare il parsing degli argomenti CLI in `bin/nyc.js` e l'implementazione di check coverage in `lib/commands/check coverage.js`. Implementa un nuovo flag ` max` che imposta tutte le soglie di copertura (righe, rami, istruzioni, funzioni) a 100 o a un valore specificato, con ` max 90` che sovrascrive solo i rami. Le soglie attuali vengono analizzate dai flag CLI; questo nuovo flag dovrebbe impostare i valori predefiniti prima di applicare eventuali flag espliciti. Rivedi la suite di test esistente per check coverage per garantire una copertura dei test adeguata.
Tech stack
javascript
Dominio
cli
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75