Skip to content

Enabled Codecov upload on 'deps/' branches to report required checks. - #695

Merged
AlexSkrypnyk merged 2 commits into
mainfrom
feature/codecov-deps
Jul 31, 2026
Merged

Enabled Codecov upload on 'deps/' branches to report required checks.#695
AlexSkrypnyk merged 2 commits into
mainfrom
feature/codecov-deps

Conversation

@AlexSkrypnyk

@AlexSkrypnyk AlexSkrypnyk commented Jul 30, 2026

Copy link
Copy Markdown
Member

Summary

The "Upload code coverage reports to Codecov" step in .github/workflows/test.yml skipped running whenever the head branch started with deps/, which is exactly the branch pattern Renovate always uses for its dependency-update PRs. Because the upload never ran on those branches, Codecov never posted the codecov/patch and codecov/project contexts, both of which the main branch ruleset lists as required status checks. Every Renovate PR therefore sat at mergeStateStatus: BLOCKED indefinitely: even with all 14 GitHub Actions checks green, the two Codecov contexts stayed at "Expected - Waiting for status to be reported" forever, so Renovate automerge could never fire. This change removes the deps/-branch exclusion so the upload runs unconditionally, matching how the sibling repos drevops/behat-screenshot and AlexSkrypnyk/drupal_helpers already configure this step.

Changes

  • Removed the && !startsWith(github.head_ref, 'deps/') clause from the if: condition on the Codecov upload step in .github/workflows/test.yml, so it now runs whenever matrix.php_version == '8.3' && matrix.drupal_version == '11' && matrix.deps == 'normal' && env.CODECOV_TOKEN != '', regardless of branch.
  • Added a comment above the condition explaining why the upload must run unconditionally, since the two Codecov contexts are required status checks and skipping them leaves a PR permanently blocked.

Before / After

┌─────────────────────────┬─────────────────────────────┬─────────────────────┐
│ Aspect (deps/* branch)  │ Before                      │ After               │
├─────────────────────────┼─────────────────────────────┼─────────────────────┤
│ GitHub Actions checks   │ 14/14 green                 │ 14/14 green         │
│ Codecov upload step     │ SKIPPED (deps/* excluded)   │ RUNS (no exclusion) │
│ codecov/patch context   │ never reported              │ reported (green)    │
│ codecov/project context │ never reported              │ reported (green)    │
│ Required status checks  │ stuck: "Expected - waiting" │ satisfied           │
│ mergeStateStatus        │ BLOCKED                     │ CLEAN               │
│ Renovate automerge      │ never fires                 │ fires               │
└─────────────────────────┴─────────────────────────────┴─────────────────────┘

Summary

  • Updated the Codecov upload step to run on all branches, including deps/*.
  • Ensures required Codecov status checks are reported for dependency-update PRs, allowing Renovate automerge.
  • Added an explanatory comment documenting the branch ruleset requirement.

Validation

  • No CONTRIBUTING.md step-definition rules apply to this GitHub Actions workflow change.

@coderabbitai

coderabbitai Bot commented Jul 30, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

You’ve reached a temporary PR review limit under our Fair Usage Limits Policy.

Your recent review volume is higher than typical usage, so adaptive limits are currently applied.

Next review available in: 26 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: f33f18d9-d566-47f6-adb7-02bb5069228f

📥 Commits

Reviewing files that changed from the base of the PR and between 3c6ff61 and abc5bc5.

📒 Files selected for processing (1)
  • .github/workflows/test.yml

Walkthrough

The workflow now uploads Codecov coverage from qualifying PHP 8.3 and Drupal 11 matrix jobs on dependency branches when CODECOV_TOKEN is available.

Changes

Codecov upload condition

Layer / File(s) Summary
Allow dependency branch uploads
.github/workflows/test.yml
The Codecov condition removes the deps/ branch exclusion while retaining the existing qualifying matrix and token checks.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Poem

I’m a bunny with coverage to share,
Sending reports through the CI air.
Dependency branches join the parade,
When a Codecov token’s arrayed.
Hop, hop—more green squares grow!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes enabling Codecov uploads on deps/ branches, matching the pull request's primary change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/codecov-deps

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Jul 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.29%. Comparing base (4fdd3cc) to head (abc5bc5).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #695   +/-   ##
=======================================
  Coverage   97.29%   97.29%           
=======================================
  Files          52       52           
  Lines        4439     4439           
=======================================
  Hits         4319     4319           
  Misses        120      120           

☔ 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.

@AlexSkrypnyk AlexSkrypnyk added the Needs review Pull request needs a review from assigned developers label Jul 30, 2026
@AlexSkrypnyk
AlexSkrypnyk merged commit 3ec72c8 into main Jul 31, 2026
15 of 16 checks passed
@AlexSkrypnyk
AlexSkrypnyk deleted the feature/codecov-deps branch July 31, 2026 00:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs review Pull request needs a review from assigned developers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant