Track 30 coverage runs with cumulative artifacts - #8500
Merged
Amaury Chamayou (achamayou) merged 4 commits intoOct 5, 2026
Merged
Conversation
Co-authored-by: achamayou <4016369+achamayou@users.noreply.github.com>
Co-authored-by: achamayou <4016369+achamayou@users.noreply.github.com>
Amaury Chamayou (achamayou)
marked this pull request as ready for review
October 2, 2026 20:35
Co-authored-by: achamayou <4016369+achamayou@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Warning
Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.
Copilot review overview
Review effort: Lite
Findings: 2
Open (3)
concurrencyconfiguration keyqueueis not part of the GitHub Actions concurrency schema… · Newcurrentis typed asOptional[CoveragePoint]but is appended unconditionally. Ifcurrentis… · New
What changed in this PR
Updates coverage trend generation to use per-branch artifacts (instead of scraping previous run logs), extends history retention to 30 runs, and adds workflow-level concurrency to serialize runs per ref.
Changes:
- Switch coverage history source from previous run logs to stored artifacts and update parsing expectations to
.txtreports. - Increase retained history/trend window to 30 runs and trim uploaded history accordingly.
- Add concurrency grouping per workflow/ref and upload an updated cumulative history artifact each successful run.
| File | Description |
|---|---|
| scripts/coverage_summary.py | Update history parsing/loading to consume stored report files and keep a 30-run trend window. |
| .github/workflows/coverage.yml | Add concurrency, restore prior history from artifacts, append current report, trim to 30, and re-upload. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Copilot started reviewing on behalf of
Amaury Chamayou (achamayou)
October 2, 2026 21:31
View session
Amaury Chamayou (achamayou)
approved these changes
Oct 3, 2026
Amaury Chamayou (achamayou)
approved these changes
Oct 5, 2026
Max (maxtropets)
approved these changes
Oct 5, 2026
Amaury Chamayou (achamayou)
deleted the
copilot/adjust-coverage-workflow
branch
October 5, 2026 09:36
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Coverage currently fetches prior job logs to build a short trend and lacks the bencher workflow’s concurrency policy. This change uses artifacts to reduce GitHub API calls and extends the trend to 30 runs.