Skip to content

Carryforward self-hosted-large coverage#2439

Open
Dreamsorcerer wants to merge 1 commit into
mainfrom
Dreamsorcerer-patch-3
Open

Carryforward self-hosted-large coverage#2439
Dreamsorcerer wants to merge 1 commit into
mainfrom
Dreamsorcerer-patch-3

Conversation

@Dreamsorcerer

Copy link
Copy Markdown
Collaborator

No description provided.

@greptile-apps

greptile-apps Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds a SelfHosted-Large carryforward flag to .codecov.yml, matching the existing pattern used for SelfHosted-Linux and SelfHosted-macos. This ensures that coverage data from self-hosted large runners is preserved across PRs that don't run those jobs.

  • Adds SelfHosted-Large under flag_management.individual_flags with carryforward: true, identical in structure to the two existing self-hosted flags.
  • No other configuration is modified; the comment at line 11 already describes the intent of this section.

Confidence Score: 5/5

Safe to merge — single-line addition to a CI configuration file with no functional side-effects.

The change adds one flag entry to Codecov configuration, exactly mirroring the two existing self-hosted flag entries. There is no logic, no code paths, and no risk of regressions.

No files require special attention.

Important Files Changed

Filename Overview
.codecov.yml Adds SelfHosted-Large flag with carryforward: true, consistent with existing SelfHosted-Linux and SelfHosted-macos flags

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[PR Coverage Upload] --> B{Flag Type}
    B --> C[SelfHosted-Linux\ncarryforward: true]
    B --> D[SelfHosted-macos\ncarryforward: true]
    B --> E[SelfHosted-Large\ncarryforward: true NEW]
    C --> F[Codecov: preserve last known coverage if job not run on PR]
    D --> F
    E --> F
Loading

Reviews (1): Last reviewed commit: "Carryforward self-hosted-large coverage" | Re-trigger Greptile

@codecov

codecov Bot commented Jun 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@github-actions github-actions Bot added the ready-to-merge Required CI checks have passed on this PR label Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Required CI checks have passed on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant