Skip to content

Try to get rid of noise in job logs from vitest#6686

Draft
npaun wants to merge 1 commit intomainfrom
npaun/get-rid-of-vitest-summary
Draft

Try to get rid of noise in job logs from vitest#6686
npaun wants to merge 1 commit intomainfrom
npaun/get-rid-of-vitest-summary

Conversation

@npaun
Copy link
Copy Markdown
Member

@npaun npaun commented Apr 28, 2026

No description provided.

@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk Bot commented Apr 28, 2026

The change is straightforward. Three steps get GITHUB_STEP_SUMMARY: /tmp/suppress-summary.txt added to their env blocks to redirect vitest's job summary output to a temp file (effectively suppressing it).

I notice the "Run Wrangler unit tests" step (line 221-227) and "Run Wrangler Node.js e2e tests" step (line 247-250) don't get this treatment. The Wrangler unit tests also use vitest, so this could be an intentional omission (maybe wrangler tests don't generate the noisy summary) or an oversight. But this is a minor CI tweak and the author likely knows which steps produce the noise. The change is safe and non-destructive.

LGTM

github run

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.

1 participant