istanbuljs/nyc

--all should report which files it failed to instrument

Open

#175 opened on Feb 28, 2016

View on GitHub
 (2 comments) (1 reaction) (0 assignees)JavaScript (5,468 stars) (392 forks)batch import
Great First Contributionenhancementhelp wantedneeds-investigation

Description

#173 uses --all on a file containing unrecognized syntax. Requiring babel-core/register is the correct solution but it's unfortunate that nyc spits out a rather deep stack trace. Instead it should report which file it failed to instrument, offering suggestions regarding excluding those files or hooking up a transpiler.

Contributor guide

Tech stack
javascriptnodejs
Domain
clitesting
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
mostly clear
Prerequisites
basic knowledge of nycfamiliarity with error handling
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70
Research direction
Investigate the codebase to locate where ` all` flag is processed and where instrumentation failures occur. The current behavior outputs a deep stack trace; the goal is to catch the error and display the file that failed, along with suggestions (e.g., excluding the file or using a transpiler). Look for relevant files in the nyc repository, possibly in the main CLI logic or the instrumentation module.