-
Notifications
You must be signed in to change notification settings - Fork 37
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
Goal: get to 100% code coverage for ArrayInterface.jl #102
Comments
What's the best way to test the parses depending on CUDA? |
We can use Buildkite, which provides GPU CI to packages in the Julia GPU ecosystem. I can take care of that. |
@chriselrod Do you have admin permissions on this repo? |
Yes. Or at least, I can merge. What needs to happen? |
Kind of complicated. We need:
|
I'll give you temporary ownership in SciML for it. |
Thanks. I'll ping you on GitHub (or Slack, if Slack comes back up) when it's done. |
Actually I don't need temporary ownership. The GitHub app is already done. Either @chriselrod or I needs temporary admin-ship on this repo only. |
Just chiming in to reassure that I will keep adding tests for code I've contributed to. |
Webhooks are added. @ChrisRackauckas Can you de-owner me when you get a chance? I'll make a PR in a bit to add the BuildKite YAML to this repo, and then we should be all set. |
De-owned. I'll keep you and @chriselrod in the team for this repo though. |
For context, here is the current code coverage:
cc: @chriselrod
The text was updated successfully, but these errors were encountered: