Skip to content

ci(coverage): use informational-patch input instead of sed hack#544

Merged
BYK merged 1 commit intomainfrom
ci/codecov-informational-input
Mar 24, 2026
Merged

ci(coverage): use informational-patch input instead of sed hack#544
BYK merged 1 commit intomainfrom
ci/codecov-informational-input

Conversation

@BYK
Copy link
Member

@BYK BYK commented Mar 24, 2026

Follow-up to #541. Replaces the sed-based codecov.yml patching with the new informational-patch action input from getsentry/codecov-action (merged in getsentry/codecov-action#54).

Changes

  • ci.yml: Remove the sed step, add informational-patch: ${{ github.event_name == 'push' }} to the codecov action
  • codecov.yml: Remove the patch.default.informational section (no longer needed)

Behavior (unchanged)

Context codecov/project codecov/patch
PR informational blocking
main / release/** push informational informational

@github-actions
Copy link
Contributor

github-actions bot commented Mar 24, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


Internal Changes 🔧

Coverage

  • Use informational-patch input instead of sed hack by BYK in #544
  • Make checks informational on release branches by BYK in #541

Other

  • (ci) Upgrade GitHub Actions to Node 24 runtime by BYK in #542
  • Regenerate skill files by github-actions[bot] in ec1ffe28

🤖 This preview updates automatically when you update the PR.

Replace the sed-based codecov.yml patching with the new
informational-patch action input from getsentry/codecov-action.

On push events (main/release branches), informational-patch evaluates
to 'true', making the patch check advisory. On PRs it evaluates to
'false', keeping it blocking.
@BYK BYK force-pushed the ci/codecov-informational-input branch from 40305c0 to 0229435 Compare March 24, 2026 00:51
@github-actions
Copy link
Contributor

Codecov Results 📊

126 passed | Total: 126 | Pass Rate: 100% | Execution Time: 0ms

📊 Comparison with Base Branch

Metric Change
Total Tests
Passed Tests
Failed Tests
Skipped Tests

✨ No test changes detected

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 1028 uncovered lines.
✅ Project coverage is 96.02%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    96.02%    96.02%        —%
==========================================
  Files          185       185         —
  Lines        25805     25805         —
  Branches         0         0         —
==========================================
+ Hits         24777     24777         —
- Misses        1028      1028         —
- Partials         0         0         —

Generated by Codecov Action

@BYK BYK merged commit b590472 into main Mar 24, 2026
22 checks passed
@BYK BYK deleted the ci/codecov-informational-input branch March 24, 2026 00:56
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