Skip to content

fix: restore coverage workflow and README badges - #6

Merged
bbuchsbaum merged 1 commit into
mainfrom
cursor/portfolio-repair-bbuchsbaum-referent-coverage-badge-1d05
Aug 28, 2026
Merged

bbuchsbaum merged 1 commit into
mainfrom
cursor/portfolio-repair-bbuchsbaum-referent-coverage-badge-1d05

Conversation

@bbuchsbaum

@bbuchsbaum bbuchsbaum commented Aug 28, 2026 •

Copy link
Copy Markdown
Owner

Summary

Repairs the portfolio coverage-badge gate for #5 by restoring missing coverage evidence without changing package APIs or existing scientific/test gates.

  • Add .github/workflows/test-coverage.yaml using the current portfolio template: actions/checkout@v6, pinned codecov/codecov-action@fb8b358… (v7), actions/upload-artifact@v7
  • Codecov upload via OIDC (id-token: write, use_oidc: true, no CODECOV_TOKEN); fail_ci_if_error: true so a broken public Codecov link fails the gate
  • Plural Codecov inputs (files, plugins); covr coverage + Cobertura retained; overall percent printed in the job log
  • README badges for GitHub Actions test-coverage and Codecov
  • Leave quality (R CMD check / scale budgets), PCNtoolkit validation, real-cohort validation, and Starlight docs unchanged

Motivation

Issue #5 reports coverage workflow state missing on default-branch SHA cf17f20. Hosted evidence must turn green after merge.

Hosted evidence (head aea9ebb)

  • Overall coverage: 94.55%
  • test-coverage workflow: passed (OIDC Codecov upload succeeded)
  • Codecov checks: codecov/project and codecov/patch passed
  • Public Codecov badge now reports 95% for this head
  • Required R package quality matrix: passed (ubuntu release/oldrel, macOS release, Windows release) plus scale smoke budgets

Out of scope

Issue #5 intentionally left open until the portfolio monitor verifies the merged default-branch SHA and hosted badge/workflow evidence. This PR is ready for review and should not be merged by the agent.

Testing

  • Validated workflow YAML contract locally (OIDC permissions, Codecov v7 inputs, badge URLs)
  • Confirmed existing specialized workflows remain present and untouched
  • GitHub Actions on this PR: all quality + coverage checks green
Open in Web Open in Cursor 

Restore the missing portfolio coverage-badge gate with a covr-based
GitHub Actions workflow and Codecov/Actions badges, without changing
existing quality or scientific validation jobs.

Co-authored-by: bbuchsbaum <bbuchsbaum@users.noreply.github.com>
@codecov

codecov Bot commented Aug 28, 2026

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@bbuchsbaum
bbuchsbaum marked this pull request as ready for review August 28, 2026 11:34

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. Cursor Security Agent completed successfully with no findings that need human review; Cursor Bugbot was not running on this PR. No reviewers were assigned.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Router and Approver

@bbuchsbaum
bbuchsbaum merged commit 5514d4f into main Aug 28, 2026
11 checks passed
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.

2 participants