Skip to content

Fix shared-workflows repo name - #14784

Merged
rapids-bot[bot] merged 4 commits into
branch-24.02from
raydouglass-patch-1
Jan 19, 2024
Merged

Fix shared-workflows repo name#14784
rapids-bot[bot] merged 4 commits into
branch-24.02from
raydouglass-patch-1

Conversation

@raydouglass

Copy link
Copy Markdown
Contributor

Description

Fix the repo name for shared-workflows

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@raydouglass raydouglass added bug Something isn't working non-breaking Non-breaking change labels Jan 18, 2024
@raydouglass
raydouglass requested a review from a team as a code owner January 18, 2024 19:55

@bdice bdice left a comment

Copy link
Copy Markdown
Contributor

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.

@vyasr vyasr left a comment

Copy link
Copy Markdown
Contributor

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

codecov Bot commented Jan 18, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (branch-24.02@d017cf4). Click here to learn what that means.

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.
📢 Have feedback on the report? Share it here.

@bdice

bdice commented Jan 18, 2024

Copy link
Copy Markdown
Contributor

/merge

@rapids-bot
rapids-bot Bot merged commit a38fc01 into branch-24.02 Jan 19, 2024
rapids-bot Bot pushed a commit to NVIDIA/cuml that referenced this pull request Jan 20, 2024
Fix the repo name for `shared-workflows`

See: NVIDIA/cudf#14784

Authors:
  - Ray Douglass (https://github.com/raydouglass)

Approvers:
  - AJ Schmidt (https://github.com/ajschmidt8)

URL: #5723
@raydouglass
raydouglass deleted the raydouglass-patch-1 branch January 22, 2024 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants