istanbuljs/nyc

nyc check-coverage: new command flag

Open

#432 geöffnet am 1. Nov. 2016

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (5.468 Stars) (392 Forks)batch import
Great First Contributionenhancementhelp wanted

Beschreibung

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?

Contributor Guide

Tech Stack
javascriptnodejs
Domain
cli
Issue Type
feature
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
Basic understanding of CLI toolsFamiliarity with nyc
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60
Research-Richtung
Explore the nyc codebase, particularly the CLI argument parsing in `bin/nyc.js` and the check coverage implementation in `lib/commands/check coverage.js`. Implement a new ` max` flag that sets all coverage thresholds (lines, branches, statements, functions) to 100 or to a specified value, with ` max 90` overriding just branches. The current thresholds are parsed from CLI flags; this new flag should set defaults before applying any explicit flags. Review the existing test suite for check coverage to ensure proper test coverage.