Skip to content

codecov: add rocPRIM coverage flag - #8955

Closed
jainprad wants to merge 1 commit into
ROCm:developfrom
jainprad:users/jainprad/codecov-rocprim-flag
Closed

codecov: add rocPRIM coverage flag#8955
jainprad wants to merge 1 commit into
ROCm:developfrom
jainprad:users/jainprad/codecov-rocprim-flag

Conversation

@jainprad

@jainprad jainprad commented Jun 30, 2026

Copy link
Copy Markdown

Summary

  • Adds the rocPRIM project coverage status/flag (80% target) to codecov.yml, matching the other ROCm math libraries already listed (rocThrust, hipCUB, rocBLAS, rocRAND, hipRAND, rocFFT, ...).

Context

Split out of #8656 per review feedback (the codecov flag is needed but out-of-scope for that PR). rocPRIM was the only coverage-enabled component missing a codecov flag.

Test plan

  • codecov.yml is config-only; validated it parses as YAML. No build/test impact.

Made with Cursor

@jainprad @cursoragent
codecov: add rocPRIM coverage flag
d1939fb 
Add the rocPRIM project coverage status/flag (80% target) so rocPRIM coverage
uploads are tracked, matching the other ROCm math libraries already listed.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions github-actions Bot added the project: none Does not target any component label Jun 30, 2026
jainprad added a commit to jainprad/rocm-libraries that referenced this pull request Jun 30, 2026
@jainprad @cursoragent
move codecov flag and rocThrust SQLite fallback to separate PRs
69b3ed3 
Per review feedback, the codecov.yml rocPRIM flag and the rocThrust SQLite3
fallback are out-of-scope for this PR. They now live in their own PRs:
- codecov rocPRIM flag: ROCm#8955
- rocThrust SQLite fallback: ROCm#8956

Note: the header-only (PRIM group) coverage build here depends on ROCm#8956
landing so rocThrust configure does not fail on find_package(SQLite3).

Co-authored-by: Cursor <cursoragent@cursor.com>
@therock-pr-bot

therock-pr-bot Bot commented Jun 30, 2026 •
edited
Loading

Copy link
Copy Markdown

❌ PR Check — Action Required

Check Status Details
🌿 Branch Name ✅ Pass —
📝 PR Title/Description ❌ Fail Error: Title does not follow Conventional Commits style.
Expected: start with a valid type (feat, fix, docs, …).
Desired format: type(optional-scope): short description
⛔ Forbidden Files ✅ Pass —
🧪 Unit Test ✅ Pass PR does not contain code files — Unit Test auto-passed
🔎 pre-commit ✅ Pass —
🚫 Draft PR 🔜 To Be Enabled —
🚩 Feature Flag 🔜 To Be Enabled —
📊 Code Coverage 🔜 To Be Enabled —

⚠️ 1 policy check(s) failed. Please address the issues above before this PR can be Reviewed.

🚫 Please fix the failed policies

  • ❌ PR Title/Description

The Not ready to Review label was added to this PR. Once all policies pass, the label is removed automatically.

📖 Need help? See the Policy FAQ for details on every check and how to fix failures.

@therock-pr-bot

therock-pr-bot Bot commented Jun 30, 2026

Copy link
Copy Markdown

🚫 Please fix the failed policies before requesting reviews.

The following policy checks failed:

  • ❌ PR Title/Description

The Not ready to Review label has been added to this PR.
Once all policies pass, the label will be removed automatically.

jainprad added a commit to jainprad/rocm-libraries that referenced this pull request Jul 21, 2026
@jainprad @cursoragent
move codecov flag and rocThrust SQLite fallback to separate PRs
c5438bd 
Per review feedback, the codecov.yml rocPRIM flag and the rocThrust SQLite3
fallback are out-of-scope for this PR. They now live in their own PRs:
- codecov rocPRIM flag: ROCm#8955
- rocThrust SQLite fallback: ROCm#8956

Note: the header-only (PRIM group) coverage build here depends on ROCm#8956
landing so rocThrust configure does not fail on find_package(SQLite3).

Co-authored-by: Cursor <cursoragent@cursor.com>
reboss pushed a commit that referenced this pull request Jul 21, 2026
@jainprad @cursoragent @reboss
move codecov flag and rocThrust SQLite fallback to separate PRs
8769fa3 
Per review feedback, the codecov.yml rocPRIM flag and the rocThrust SQLite3
fallback are out-of-scope for this PR. They now live in their own PRs:
- codecov rocPRIM flag: #8955
- rocThrust SQLite fallback: #8956

Note: the header-only (PRIM group) coverage build here depends on #8956
landing so rocThrust configure does not fail on find_package(SQLite3).

Co-authored-by: Cursor <cursoragent@cursor.com>
jainprad added a commit to jainprad/rocm-libraries that referenced this pull request Jul 21, 2026
@jainprad @cursoragent
move codecov flag and rocThrust SQLite fallback to separate PRs
867a041 
Per review feedback, the codecov.yml rocPRIM flag and the rocThrust SQLite3
fallback are out-of-scope for this PR. They now live in their own PRs:
- codecov rocPRIM flag: ROCm#8955
- rocThrust SQLite fallback: ROCm#8956

Note: the header-only (PRIM group) coverage build here depends on ROCm#8956
landing so rocThrust configure does not fail on find_package(SQLite3).

Co-authored-by: Cursor <cursoragent@cursor.com>
reboss pushed a commit that referenced this pull request Jul 23, 2026
@jainprad @cursoragent @reboss
move codecov flag and rocThrust SQLite fallback to separate PRs
b022144 
Per review feedback, the codecov.yml rocPRIM flag and the rocThrust SQLite3
fallback are out-of-scope for this PR. They now live in their own PRs:
- codecov rocPRIM flag: #8955
- rocThrust SQLite fallback: #8956

Note: the header-only (PRIM group) coverage build here depends on #8956
landing so rocThrust configure does not fail on find_package(SQLite3).

Co-authored-by: Cursor <cursoragent@cursor.com>
reboss pushed a commit that referenced this pull request Jul 23, 2026
@jainprad @cursoragent @reboss
move codecov flag and rocThrust SQLite fallback to separate PRs
507cc50 
Per review feedback, the codecov.yml rocPRIM flag and the rocThrust SQLite3
fallback are out-of-scope for this PR. They now live in their own PRs:
- codecov rocPRIM flag: #8955
- rocThrust SQLite fallback: #8956

Note: the header-only (PRIM group) coverage build here depends on #8956
landing so rocThrust configure does not fail on find_package(SQLite3).

Co-authored-by: Cursor <cursoragent@cursor.com>
reboss pushed a commit that referenced this pull request Jul 24, 2026
@jainprad @cursoragent @reboss
move codecov flag and rocThrust SQLite fallback to separate PRs
9cc06e9 
Per review feedback, the codecov.yml rocPRIM flag and the rocThrust SQLite3
fallback are out-of-scope for this PR. They now live in their own PRs:
- codecov rocPRIM flag: #8955
- rocThrust SQLite fallback: #8956

Note: the header-only (PRIM group) coverage build here depends on #8956
landing so rocThrust configure does not fail on find_package(SQLite3).

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

github-actions Bot commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

This pull request has been inactive for 25 days and will be marked as stale.

If you would like to keep this PR open, please:

  • Add new commits
  • Add a comment explaining why it should remain open

This PR will be automatically closed in 5 days if no further activity occurs.

@github-actions github-actions Bot added the Stale PR has no activity for 25+ days label Jul 26, 2026
reboss pushed a commit that referenced this pull request Jul 27, 2026
@jainprad @cursoragent @reboss
move codecov flag and rocThrust SQLite fallback to separate PRs
b21c2cc 
Per review feedback, the codecov.yml rocPRIM flag and the rocThrust SQLite3
fallback are out-of-scope for this PR. They now live in their own PRs:
- codecov rocPRIM flag: #8955
- rocThrust SQLite fallback: #8956

Note: the header-only (PRIM group) coverage build here depends on #8956
landing so rocThrust configure does not fail on find_package(SQLite3).

Co-authored-by: Cursor <cursoragent@cursor.com>
reboss pushed a commit that referenced this pull request Jul 27, 2026
@jainprad @cursoragent @reboss
move codecov flag and rocThrust SQLite fallback to separate PRs
80eaad8 
Per review feedback, the codecov.yml rocPRIM flag and the rocThrust SQLite3
fallback are out-of-scope for this PR. They now live in their own PRs:
- codecov rocPRIM flag: #8955
- rocThrust SQLite fallback: #8956

Note: the header-only (PRIM group) coverage build here depends on #8956
landing so rocThrust configure does not fail on find_package(SQLite3).

Co-authored-by: Cursor <cursoragent@cursor.com>
reboss pushed a commit that referenced this pull request Jul 29, 2026
@jainprad @cursoragent @reboss
move codecov flag and rocThrust SQLite fallback to separate PRs
77bd53a 
Per review feedback, the codecov.yml rocPRIM flag and the rocThrust SQLite3
fallback are out-of-scope for this PR. They now live in their own PRs:
- codecov rocPRIM flag: #8955
- rocThrust SQLite fallback: #8956

Note: the header-only (PRIM group) coverage build here depends on #8956
landing so rocThrust configure does not fail on find_package(SQLite3).

Co-authored-by: Cursor <cursoragent@cursor.com>
reboss pushed a commit that referenced this pull request Jul 29, 2026
@jainprad @cursoragent @reboss
move codecov flag and rocThrust SQLite fallback to separate PRs
46c913f 
Per review feedback, the codecov.yml rocPRIM flag and the rocThrust SQLite3
fallback are out-of-scope for this PR. They now live in their own PRs:
- codecov rocPRIM flag: #8955
- rocThrust SQLite fallback: #8956

Note: the header-only (PRIM group) coverage build here depends on #8956
landing so rocThrust configure does not fail on find_package(SQLite3).

Co-authored-by: Cursor <cursoragent@cursor.com>
reboss pushed a commit that referenced this pull request Jul 29, 2026
@jainprad @cursoragent @reboss
move codecov flag and rocThrust SQLite fallback to separate PRs
285274b 
Per review feedback, the codecov.yml rocPRIM flag and the rocThrust SQLite3
fallback are out-of-scope for this PR. They now live in their own PRs:
- codecov rocPRIM flag: #8955
- rocThrust SQLite fallback: #8956

Note: the header-only (PRIM group) coverage build here depends on #8956
landing so rocThrust configure does not fail on find_package(SQLite3).

Co-authored-by: Cursor <cursoragent@cursor.com>
reboss pushed a commit that referenced this pull request Jul 29, 2026
@jainprad @cursoragent @reboss
move codecov flag and rocThrust SQLite fallback to separate PRs
3e2e484 
Per review feedback, the codecov.yml rocPRIM flag and the rocThrust SQLite3
fallback are out-of-scope for this PR. They now live in their own PRs:
- codecov rocPRIM flag: #8955
- rocThrust SQLite fallback: #8956

Note: the header-only (PRIM group) coverage build here depends on #8956
landing so rocThrust configure does not fail on find_package(SQLite3).

Co-authored-by: Cursor <cursoragent@cursor.com>
reboss pushed a commit that referenced this pull request Jul 29, 2026
@jainprad @cursoragent @reboss
move codecov flag and rocThrust SQLite fallback to separate PRs
072cf5c 
Per review feedback, the codecov.yml rocPRIM flag and the rocThrust SQLite3
fallback are out-of-scope for this PR. They now live in their own PRs:
- codecov rocPRIM flag: #8955
- rocThrust SQLite fallback: #8956

Note: the header-only (PRIM group) coverage build here depends on #8956
landing so rocThrust configure does not fail on find_package(SQLite3).

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

github-actions Bot commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

This pull request has been automatically closed due to inactivity (30 days with no updates).

If you'd like to continue working on this, feel free to reopen the PR or create a new one.

@github-actions github-actions Bot closed this Aug 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

organization: ROCm project: none Does not target any component Stale PR has no activity for 25+ days

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@jainprad