Skip to content

Add devcontainer fallback for C++ test location#7596

Merged
rapids-bot[bot] merged 1 commit intorapidsai:mainfrom
bdice:ctest-devcontainer-fallback
Dec 12, 2025
Merged

Add devcontainer fallback for C++ test location#7596
rapids-bot[bot] merged 1 commit intorapidsai:mainfrom
bdice:ctest-devcontainer-fallback

Conversation

@bdice
Copy link
Copy Markdown
Contributor

@bdice bdice commented Dec 11, 2025

Summary

  • Update run_ctests.sh to first try the installed test location (CI/conda environments) and fall back to the build directory (devcontainer environments)
  • Enables testing in devcontainers with test-cuml-cpp

xref: rapidsai/devcontainers#630

Update run_ctests.sh to first try the installed test location (CI/conda
environments) and fall back to the build directory (devcontainer
environments). This allows running tests in both environments without
requiring installation.
@bdice bdice requested a review from a team as a code owner December 11, 2025 01:31
@bdice bdice requested a review from KyleFromNVIDIA December 11, 2025 01:31
@github-actions github-actions bot added the ci label Dec 11, 2025
@bdice bdice added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Dec 11, 2025
@bdice
Copy link
Copy Markdown
Contributor Author

bdice commented Dec 12, 2025

/merge

@rapids-bot rapids-bot bot merged commit 412210b into rapidsai:main Dec 12, 2025
111 checks passed
mani-builds pushed a commit to mani-builds/cuml that referenced this pull request Jan 11, 2026
## Summary
- Update `run_ctests.sh` to first try the installed test location (CI/conda environments) and fall back to the build directory (devcontainer environments)
- Enables testing in devcontainers with `test-cuml-cpp`

xref: rapidsai/devcontainers#630

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

URL: rapidsai#7596
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants