Skip to content

Compute matrix for nightly conda environment checks. - #756

Closed
bdice wants to merge 2 commits into
rapidsai:branch-25.06from
bdice:compute-matrix
Closed

Compute matrix for nightly conda environment checks.#756
bdice wants to merge 2 commits into
rapidsai:branch-25.06from
bdice:compute-matrix

Conversation

@bdice

@bdice bdice commented Apr 8, 2025

Copy link
Copy Markdown
Contributor

Currently CI is failing because we are using conda-python-build.yaml for a CPU test job. Some changes were made in rapidsai/shared-workflows#297 that make this workflow less suitable for this generalized usage. Instead, we should compute a custom matrix and use that.

@copy-pr-bot

copy-pr-bot Bot commented Apr 8, 2025

Copy link
Copy Markdown

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.

@bdice

bdice commented Apr 8, 2025

Copy link
Copy Markdown
Contributor Author

/ok to test

@bdice
bdice marked this pull request as ready for review April 8, 2025 18:53
@bdice
bdice requested a review from a team as a code owner April 8, 2025 18:53
@bdice
bdice requested a review from AyodeAwe April 8, 2025 18:53
@AyodeAwe

AyodeAwe commented Apr 8, 2025

Copy link
Copy Markdown
Contributor

There might be a syntax error in the pr.yaml workflow

@bdice

bdice commented Apr 10, 2025

Copy link
Copy Markdown
Contributor Author

@AyodeAwe Hmm. I will need to dig into this again to figure out the problem but I have a lot on my plate at the moment. If you want to push to this PR, please do!

@jameslamb

Copy link
Copy Markdown
Member

@bdice I think we can close this, given that we instead went the route of making whether or not the *-build shared workflows try to upload artifacts to GitHub Actions artifact store:

@jameslamb jameslamb closed this May 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants