Skip to content
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

Test against rmm #337

Closed
wants to merge 1 commit into from
Closed

Conversation

vyasr
Copy link
Contributor

@vyasr vyasr commented Nov 27, 2024

No description provided.

@vyasr vyasr added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Nov 27, 2024
@vyasr vyasr self-assigned this Nov 27, 2024
Copy link

copy-pr-bot bot commented Nov 27, 2024

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@vyasr
Copy link
Contributor Author

vyasr commented Nov 27, 2024

/ok to test

@vyasr
Copy link
Contributor Author

vyasr commented Dec 30, 2024

#343 uncovered that ucxx does not need the rmm logger at all, so this is a poor test case. I'll reconstruct this in another repo.

@vyasr vyasr closed this Dec 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improves an existing functionality non-breaking Introduces a non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant