Skip to content

Conversation

@robobun
Copy link
Collaborator

@robobun robobun commented Feb 10, 2026

Summary

  • Adds --reporter=dots to bun test invocations in scripts/runner.node.mjs to reduce CI log output volume
  • The dots reporter prints a single character per test (. for pass, X for fail) instead of full test names
  • When JUnit reporting is also enabled, both reporters stack correctly — dots goes to stdout, JUnit writes to file

Test plan

  • Verify CI builds run with dots reporter and produce smaller logs
  • Verify JUnit reports are still generated correctly when --junit is enabled

🤖 Generated with Claude Code

Add --reporter=dots to bun test invocations in the test runner script
to minimize output volume in CI. The dots reporter prints a single
character per test instead of full test names, significantly reducing
log size. When JUnit reporting is enabled, both reporters stack
correctly (dots to stdout, JUnit to file).

Co-Authored-By: Claude <noreply@anthropic.com>
@robobun
Copy link
Collaborator Author

robobun commented Feb 10, 2026

Updated 8:30 PM PT - Feb 9th, 2026

❌ Your commit 98bbb2e6 has 4 failures in Build #36792 (All Failures):


🧪   To try this PR locally:

bunx bun-pr 26860

That installs a local version of the PR into your bun-26860 executable, so you can run:

bun-26860 --bun

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 10, 2026

Walkthrough

The change modifies the test runner in scripts/runner.node.mjs by adding a --reporter=dots flag to the bun test arguments, altering the output format for test results.

Changes

Cohort / File(s) Summary
Test Runner Configuration
scripts/runner.node.mjs
Added --reporter=dots flag to the spawnBunTest function arguments, changing the bun test output format to use the dots reporter.
🚥 Pre-merge checks | ✅ 1 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description covers the main change and testing approach, but does not follow the required template structure with 'What does this PR do?' and 'How did you verify your code works?' sections. Restructure the description to match the template: add 'What does this PR do?' section and 'How did you verify your code works?' section with proper formatting.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Reduce CI test output by using dots reporter' directly and clearly summarizes the main change—adding a dots reporter to reduce CI log output.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


No actionable comments were generated in the recent review. 🎉


Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant