Skip to content

fix(ci): generate JUnit XML to silence codecov-action warnings#300

Merged
BYK merged 2 commits intomainfrom
fix/junit-xml-ci-warnings
Feb 26, 2026
Merged

fix(ci): generate JUnit XML to silence codecov-action warnings#300
BYK merged 2 commits intomainfrom
fix/junit-xml-ci-warnings

Conversation

@BYK
Copy link
Member

@BYK BYK commented Feb 26, 2026

Summary

getsentry/codecov-action@main has enable-tests: true by default, searching for JUnit XML files matching **/*.junit.xml. Since the project produced no JUnit output, every CI run emitted 3 warnings about missing JUnit XML files.

Bun's test runner natively supports JUnit XML via bunfig.toml, so enabling it is the cleanest fix — it also unlocks test result reporting (test counts, pass rate, test deltas vs base branch) in the codecov action for free.

Changes

  • bunfig.toml: added [test.reporter] section with junit = "test-results.junit.xml"
  • .gitignore: added *.junit.xml under test artifacts so the generated file isn't accidentally committed

No CI workflow changes needed — the codecov action's default glob **/*.junit.xml already matches the output path.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 26, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


Bug Fixes 🐛

  • (ci) Generate JUnit XML to silence codecov-action warnings by BYK in #300
  • (nightly) Push to GHCR from artifacts dir so layer titles are bare filenames by BYK in #301

🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Contributor

Codecov Results 📊

2071 passed | Total: 2071 | Pass Rate: 100% | Execution Time: 0ms

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 3607 uncovered lines.
✅ Project coverage is 76.42%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    76.42%    76.42%        —%
==========================================
  Files          117       117         —
  Lines        15298     15298         —
  Branches         0         0         —
==========================================
+ Hits         11691     11691         —
- Misses        3607      3607         —
- Partials         0         0         —

Generated by Codecov Action

@BYK BYK force-pushed the fix/junit-xml-ci-warnings branch from 7d6b8c9 to f13f2e6 Compare February 26, 2026 15:47
@BYK BYK marked this pull request as ready for review February 26, 2026 15:58
@BYK BYK merged commit e36775a into main Feb 26, 2026
17 checks passed
@BYK BYK deleted the fix/junit-xml-ci-warnings branch February 26, 2026 15:59
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

<!-- lore:019c91e0-6d82-7f46-916a-f1e6555b8175 -->
* **General coding preference**: Prefer explicit error handling over silent failures
<!-- lore:019c91e0-6ce0-740b-b448-9fa0f31fedaf -->
* **Code style**: User prefers no backwards-compat shims, fix callers directly
Copy link

Choose a reason for hiding this comment

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

Duplicate preference entries added to AGENTS.md

Low Severity

Two preference entries are duplicated verbatim with different lore IDs. "General coding preference: Prefer explicit error handling over silent failures" appears on both lines 655 and 659, and "Code style: User prefers no backwards-compat shims, fix callers directly" appears on both lines 657 and 661. Since AGENTS.md is consumed as context by AI coding agents, redundant entries waste context window budget and add noise.

Additional Locations (1)

Fix in Cursor Fix in Web

betegon pushed a commit that referenced this pull request Feb 26, 2026
## Summary

`getsentry/codecov-action@main` has `enable-tests: true` by default,
searching for JUnit XML files matching `**/*.junit.xml`. Since the
project produced no JUnit output, every CI run emitted 3 warnings about
missing JUnit XML files.

Bun's test runner natively supports JUnit XML via `bunfig.toml`, so
enabling it is the cleanest fix — it also unlocks test result reporting
(test counts, pass rate, test deltas vs base branch) in the codecov
action for free.

## Changes

- **`bunfig.toml`**: added `[test.reporter]` section with `junit =
"test-results.junit.xml"`
- **`.gitignore`**: added `*.junit.xml` under test artifacts so the
generated file isn't accidentally committed

No CI workflow changes needed — the codecov action's default glob
`**/*.junit.xml` already matches the output path.
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