Skip to content

tests: refresh sample artifacts#16916

Merged
paulirish merged 3 commits into
mainfrom
updatesamplev2
Mar 20, 2026
Merged

tests: refresh sample artifacts#16916
paulirish merged 3 commits into
mainfrom
updatesamplev2

Conversation

@paulirish

@paulirish paulirish commented Mar 19, 2026

Copy link
Copy Markdown
Member

Wouldn't recommend opening any of the artifact/sample JSON diffs :)

This was a refresh of the sample artifacts and report JSONs to keep them current with the latest audits and strings.

Doing this means we'll have results in our sample report for the baseline audit! 🔥

i did this. plus my agent doing fixes.

  yarn update:sample-artifacts
  yarn update:sample-json

Also included:

  • Updated unit test snapshots (CSV, CLS precision, scoring calculator links) that were invalidated by the fresh data.
  • Fixed a Node 22 deprecation in cli/test/cli/printer-test.js where fs.rmdirSync(..., {recursive: true}) is no longer supported.

previously: #16322, #15962

@paulirish paulirish requested a review from a team as a code owner March 19, 2026 21:20
@paulirish paulirish requested review from connorjclark and removed request for a team March 19, 2026 21:20
@paulirish paulirish changed the title test: refresh sample artifacts tests: refresh sample artifacts Mar 19, 2026
@paulirish paulirish requested a review from lusayaa March 19, 2026 21:21
@paulirish paulirish enabled auto-merge (squash) March 20, 2026 00:10
@paulirish paulirish merged commit 1b51cc9 into main Mar 20, 2026
31 checks passed
@paulirish paulirish deleted the updatesamplev2 branch March 20, 2026 14:05
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.

2 participants