Skip to content

Report updated snapshot files#3446

Closed
DailyXplorer wants to merge 1 commit into
avajs:mainfrom
DailyXplorer:report-snapshot-updates
Closed

Report updated snapshot files#3446
DailyXplorer wants to merge 1 commit into
avajs:mainfrom
DailyXplorer:report-snapshot-updates

Conversation

@DailyXplorer
Copy link
Copy Markdown

@DailyXplorer DailyXplorer commented May 12, 2026

Summary

  • Track updated snapshot files from existing touched-files events in run status
  • Show the updated snapshot file count in the default reporter summary
  • Add a focused reporter test that verifies .snap files are counted and .md reports are ignored

Contributes to #2501 by addressing the snapshot update reporting item from #1583.

Tests

  • npm test
  • npx tap test-tap/reporters/default.js
  • npx tap test-tap/api.js
  • npx tap test-tap/reporters/tap.js
  • npx test-ava 'test/snapshot-workflow/*.js'
  • npx xo lib/run-status.js lib/reporters/default.js test-tap/reporters/default.js

IssueHunt Summary

Referenced issues

This pull request has been submitted to:


@sindresorhus
Copy link
Copy Markdown
Member

We don't accept fully AI-generated PRs.

@avajs avajs locked as spam and limited conversation to collaborators May 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants