Skip to content

feat(ci): delta-count summaries to containers workflow - #375

Open
wdconinc with Copilot wants to merge 1 commit into
masterfrom
copilot/eic-apply-pr-2786
Open

feat(ci): delta-count summaries to containers workflow#375
wdconinc with Copilot wants to merge 1 commit into
masterfrom
copilot/eic-apply-pr-2786

Conversation

Copilot AI commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

This ports the capybara reporting behavior from eic/EICrecon#2786 into eic/containers so comparison output is retained in artifacts and PR summaries show per-comparison diff signal counts.

  • Captured capybara output per comparison

    • Updated all capybara comparison blocks in .github/workflows/build-push.yml to stream stderr/stdout into each report directory as capybara.log via tee, instead of only creating hidden marker files.
  • Made PR summary show comparison deltas

    • Updated the “Populate capybara summary” step to compute and append (Δ: N) for each comparison by counting matching diff-stat lines (p = / p_KS =) from that comparison’s capybara.log.
  • Scope

    • Applied consistently across previous-run comparisons and amd64-vs-arm64 comparison jobs, preserving existing artifact/report structure while enriching report content.
capybara bara <ref-artifact> <new-artifact> \
  2>&1 | tee capybara-reports/capybara.log

count=$(grep -cE "p(_KS)? = " "${dir}/capybara.log" 2>/dev/null) || count=0
line="${line} (Δ: ${count})"

Copilot AI review requested due to automatic review settings July 31, 2026 00:27

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Copilot AI changed the title ci: apply EICrecon#2786 capybara logging and summary updates CI: port EICrecon#2786 capybara logging + delta-count summaries to containers workflow Jul 31, 2026
Copilot AI requested a review from wdconinc July 31, 2026 00:27
@wdconinc
wdconinc marked this pull request as ready for review July 31, 2026 00:31
@wdconinc wdconinc changed the title CI: port EICrecon#2786 capybara logging + delta-count summaries to containers workflow feat(ci): delta-count summaries to containers workflow Jul 31, 2026
@github-actions

Copy link
Copy Markdown

Capybara summary for PR 375

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.

3 participants