-
Notifications
You must be signed in to change notification settings - Fork 915
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix shared-workflows repo name #14784
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Can you also remove -DNVBench_ENABLE_CUPTI=OFF
from the devcontainer job while we're here? That is no longer needed. Otherwise we can file a separate PR for that, but it feels like a small enough change to wrap into another PR and conserve CI resources.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I took care of removing the cupti flag.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## branch-24.02 #14784 +/- ##
===============================================
Coverage ? 86.05%
===============================================
Files ? 175
Lines ? 27441
Branches ? 0
===============================================
Hits ? 23614
Misses ? 3827
Partials ? 0 ☔ View full report in Codecov by Sentry. |
/merge |
Fix the repo name for `shared-workflows` See: rapidsai/cudf#14784 Authors: - Ray Douglass (https://github.com/raydouglass) Approvers: - AJ Schmidt (https://github.com/ajschmidt8) URL: #5723
Description
Fix the repo name for
shared-workflows
Checklist