Skip to content

fix(coverage): warn instead of erroring when source files are missing - #32398

Merged
bartlomieju merged 1 commit into
denoland:mainfrom
bartlomieju:fix/coverage-missing-source-warning
Mar 2, 2026
Merged

bartlomieju merged 1 commit into
denoland:mainfrom
bartlomieju:fix/coverage-missing-source-warning

Conversation

@bartlomieju

Copy link
Copy Markdown
Member

When a source file is deleted after coverage was collected (e.g. a build artifact cleaned up before the report runs), deno coverage now prints a warning and skips the file instead of failing the entire report.

Closes #32170

When a source file is deleted after coverage was collected (e.g. a
build artifact cleaned up before the report runs), `deno coverage`
now prints a warning and skips the file instead of failing the
entire report.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

@marvinhagemeister marvinhagemeister left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bartlomieju
bartlomieju merged commit 84f7a15 into denoland:main Mar 2, 2026
112 checks passed
@bartlomieju
bartlomieju deleted the fix/coverage-missing-source-warning branch March 4, 2026 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error generating coverage report: Failed to fetch .... from cache

2 participants