refactor: switch to rapids-artifact-name for consistent artifact naming#642
Draft
gforsyth wants to merge 2 commits into
Draft
refactor: switch to rapids-artifact-name for consistent artifact naming#642gforsyth wants to merge 2 commits into
gforsyth wants to merge 2 commits into
Conversation
gforsyth
added
improvement
gforsyth
force-pushed
the
rapids-artifact-name
branch
from
2f8a3a5 to
eb2e167
Compare
rapids-bot Bot
pushed a commit
to rapidsai/gha-tools
that referenced
this pull request
Apr 29, 2026
…a-dependent pure artifacts (#254) xref rapidai/build-planning#270 Working on rapidsai/ucxx#642 which raised the issue of our different types of "pure" artifacts -- those that are truly Python-only and so have no cuda or architecture dependencies, and those that are agnostic to Python version, but still have cuda dependencies. Adding handling here, so that passing `--pure` without `--cuda` will elide the `{arch}` part of the artifact name. If we use `--pure` and `--cuda`, then `{arch}` will be included. If `--pure` isn't used, `{arch}` is always included. Authors: - Gil Forsyth (https://github.com/gforsyth) Approvers: - Kyle Edwards (https://github.com/KyleFromNVIDIA) URL: #254
|
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. |
gforsyth
force-pushed
the
rapids-artifact-name
branch
from
eb2e167 to
fa577ca
Compare
Contributor
Author
|
/ok to test |
1 similar comment
Contributor
Author
|
/ok to test |
gforsyth
force-pushed
the
rapids-artifact-name
branch
from
4dcf60d to
586f87d
Compare
Contributor
Author
|
/ok to test |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR swaps in
rapids-artifact-nameforrapids-package-nameeverywhere, and also removes any legacy named artifacts. All artifacts now follow the same naming convention (and that convention can be updated/expanded from a central location). Part of rapidsai/build-planning#2370