Skip to content

ci: clarify Octocov report freshness and test duration - #9432

Merged
vitormattos merged 1 commit into
mainfrom
ci/octocov-report-context-and-timing
Nov 10, 2026
Merged

vitormattos merged 1 commit into
mainfrom
ci/octocov-report-context-and-timing

Conversation

@vitormattos

Copy link
Copy Markdown
Member

Purpose

Keep Octocov's native coverage tables and its single updated comment while making the scope of each report explicit. A newer commit with pending, canceled, or failed PHPUnit checks may not yet have a refreshed report.

Change

  • Add comment.message as a short note above Octocov's native metrics table. This is additive in the currently pinned Octocov v0.83.5 implementation (createReportContent appends message before the metrics tables).
  • Set testExecutionTime.steps: [PHPUnit] so the duration tracks the actual PHPUnit step across the source workflow, instead of the brief Octocov collection step.
  • Keep comment.updatePrevious: true, the existing job summary, report and diff configuration intact.

Scope

Only .octocov.yml changes. No tests or business logic from #9429 are changed, and this PR does not alter the source coverage workflow or make partial/failed coverage look successful.

Validation

  • Parsed updated YAML with PyYAML; asserted testExecutionTime.steps, the additional comment.message and preservation of updatePrevious, summary, report and diff.
  • Inspected upstream Octocov v0.83.5 source, verifying comment.message is appended to (not substituted for) the native report; checked testExecutionTime.steps semantics against upstream documentation.
  • Git commit is cryptographically verified and includes DCO sign-off.

CI publishing behavior still needs confirmation in GitHub Actions; no full PHPUnit suite was run for this configuration-only change.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos
vitormattos requested a review from a team as a code owner November 10, 2026 15:20
@vitormattos
vitormattos merged commit d989d5a into main Nov 10, 2026
66 checks passed
@vitormattos
vitormattos deleted the ci/octocov-report-context-and-timing branch November 10, 2026 15:36
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