Skip to content

Conversation

@martincostello
Copy link
Collaborator

Disable including passed tests to fix the step summaries being too large since the upgrade to GitHubActionsTestLogger v3.

Disable including passed tests to fix the step summaries being too large since the upgrade to GitHubActionsTestLogger v3.
@codecov
Copy link

codecov bot commented Nov 30, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.66%. Comparing base (243201c) to head (a0a154a).
⚠️ Report is 1 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3691   +/-   ##
=======================================
  Coverage   94.66%   94.66%           
=======================================
  Files         111      111           
  Lines        3858     3858           
  Branches      778      778           
=======================================
  Hits         3652     3652           
  Misses        206      206           
Flag Coverage Δ
Linux 94.66% <ø> (ø)
Windows 94.66% <ø> (ø)
macOS 94.66% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@martincostello martincostello marked this pull request as ready for review November 30, 2025 20:10
Copilot AI review requested due to automatic review settings November 30, 2025 20:10
@martincostello martincostello enabled auto-merge (rebase) November 30, 2025 20:11
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes GitHub step summaries being too large after upgrading to GitHubActionsTestLogger v3 by disabling the inclusion of passed tests in the summaries.

  • Added summary-include-passed=false parameter to the GitHubActions test logger configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@martincostello martincostello merged commit 4ae817b into master Nov 30, 2025
25 of 26 checks passed
@martincostello martincostello deleted the fix-test-summaries branch November 30, 2025 20:25
This was referenced Dec 12, 2025
This was referenced Dec 19, 2025
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.

2 participants