Skip to content

Reimplement wheel-ctk-name-gen as a gha-tool #77

Closed

Description

This repository currently has a composite action wheel-ctk-name-gen. That action currently sets two environment variables into the GITHUB_ENV so that they are accessible in subsequent steps. This logic should instead be extracted into a new script in the gha-tools repo. Rather than setting two environment variables, we should remove the RAPIDS_PY_WHEEL_NAME altogether and simply return the suffix rather than storing it in the environment.

I also think that we should probably have the new tool return the suffix without the hyphen (e.g. cu11 instead of -cu11). The current approach leads to different parts of the workflow needing to edit that to either remove the hyphen or convert it to an underscore, so we would be better served by removing that from the initial definition of the suffix and just tacking on the extra characters where needed.

I expect that making this change will also necessitate combining more steps in the workflow since we will no longer be automatically propagating the variable across multiple steps.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions