microsoft / microsoft/vscode-test-cli

Expose c8 excludeAfterRemap argument to coverage config

Open
#37 1 comment 0 reactions 1 assignee View on GitHub

@connor4312 is already working on this.

Since Apr 15, 2024.

bug
Dominant language
TypeScript
Stars
53
Forks
16
Avg merge
10h 57m
Merged PRs (30d)
2

Description

We have a project which runs tests using the CLI against a webpacked extension and the node_modules aren't correctly excluded from coverage stats even if it's done explicitly. Manually bodging vscode-test-cli to add excludeAfterRemap: true to the Report in coverage.mts allows it to be successfully excluded. Could we get this exposed as an option or set to true by default?

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.