-
Notifications
You must be signed in to change notification settings - Fork 16
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
New script to get latest conda artifact from a given repo PR #85
Conversation
Can someone /ok to test this? |
/ok to test |
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.
blocking merge while I leave some feedback
The feedback I would've left would've touched the entire file. So I opted to just push a commit instead. Hope you don't mind Mark. The commit I pushed consists of the following changes:
|
Thanks @ajschmidt8! |
We recently added some functionality in rapidsai/gha-tools#85 that depends on the `gh` CLI, which depends on the `GH_TOKEN` variable being set in GHAs. This PR adds the `GH_TOKEN` to our test workflows. It's already included in our build workflows.
I found myself creating a bunch of PRs to test downstream affects of an RMM PR on various RAPIDS repos. Each of these required a lot of boiler plate like the following to compute the path to a CI artifact from RMM and other dependent PRs (e.g. cuML depends on RMM and RAFT and CUDF).
This PR adds a new gha-tools script to eliminate a lot of this boilerplate. Now you can do like this: