Skip to content

trunk-merge/pr-1278/d182110c-6083-41a0-8efa-ac6031a139df - #1291

Closed
trunk-io[bot] wants to merge 2 commits into
mainfrom
trunk-merge/pr-1278/d182110c-6083-41a0-8efa-ac6031a139df
Closed

trunk-io[bot] wants to merge 2 commits into
mainfrom
trunk-merge/pr-1278/d182110c-6083-41a0-8efa-ac6031a139df

Conversation

@trunk-io

@trunk-io trunk-io Bot commented Nov 4, 2026

Copy link
Copy Markdown
Trunk Merge Pull Request Banner

This pull request was created and is being managed by Trunk Merge.

This pull request is based on the main branch at SHA b66484586c58288bc3ddb05b4e33910dd52c40f5.

See more details here.

When CI completes, this pull request will be closed automatically.

Pull Requests Being Tested

This pull request is testing the changes from pull request 1278.

dfrankland and others added 2 commits November 4, 2026 20:19
Trunk drops every report that has old (> 24 hour) timestamps when it
ingests an upload. The summary only said so in each file's warnings,
which the file limit cuts off, so a run could show passing tests that
Trunk then dropped.

The upload and validate summaries now say how many files have old
timestamps, that Trunk will drop their results, and that the test
counts include them. They also suggest --bazel-bep-path, which skips
results that Bazel restored from its cache. The line is never truncated
and shows with --validation-report none too. The JSON summary gets
filesWithOldTimestamps and a per-file oldTimestamps, and
github-annotations adds a warning. Exit codes and uploads don't change.

Refs TRUNK-23000

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@trunk-staging-io

trunk-staging-io Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

Static Badge   Static Badge   Static Badge

Failed Test Failure Summary Logs
pending_quarantine_test should be quarantined when run with variant A test marked as pending was expected to fail but unexpectedly passed. Logs ↗︎
variant_quarantine_test should be quarantined when run with variant The quarantine test failed because it expected 2 + 2 to equal 5, but the result was 4. Logs ↗︎

View Full Report ↗︎ ⋅ Docs

@codecov-commenter

codecov-commenter commented Nov 4, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.22642% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.05%. Comparing base (b664845) to head (c357140).

Files with missing lines Patch % Lines
cli/src/summary_output/text.rs 92.30% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1291      +/-   ##
==========================================
+ Coverage   85.81%   86.05%   +0.24%     
==========================================
  Files          85       85              
  Lines       19395    19447      +52     
==========================================
+ Hits        16643    16735      +92     
+ Misses       2752     2712      -40     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@trunk-io trunk-io Bot closed this Nov 4, 2026
@trunk-io
trunk-io Bot deleted the trunk-merge/pr-1278/d182110c-6083-41a0-8efa-ac6031a139df branch November 4, 2026 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants