Skip to content

Update RAPIDS.cmake to log source of rapids-cmake#155

Open
arhag23 wants to merge 1 commit into
rapidsai:mainfrom
arhag23:rapids-cmake-log-source
Open

Update RAPIDS.cmake to log source of rapids-cmake#155
arhag23 wants to merge 1 commit into
rapidsai:mainfrom
arhag23:rapids-cmake-log-source

Conversation

@arhag23

@arhag23 arhag23 commented Jul 6, 2026

Copy link
Copy Markdown

@arhag23 arhag23 requested a review from a team as a code owner July 6, 2026 19:57
@github-actions github-actions Bot added the CMake label Jul 6, 2026
@coderabbitai

coderabbitai Bot commented Jul 6, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 04fd0183-7718-4c40-aa8a-11b5f3225568

📥 Commits

Reviewing files that changed from the base of the PR and between 4e9e382 and 683b733.

📒 Files selected for processing (1)
  • cmake/RAPIDS.cmake

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Added status messages during setup to show where rapids-cmake is being retrieved from, including both repository and archive-based download paths.

Walkthrough

This change adds two message(STATUS ...) log statements to cmake/RAPIDS.cmake, reporting the fetch source for rapids-cmake in both the git-based fetch branch and the archive URL fetch branch. No control flow or fetch logic was altered.

Changes

Fetch logging

Layer / File(s) Summary
Add status logging for rapids-cmake fetch source
cmake/RAPIDS.cmake
Adds message(STATUS ...) calls reporting the fetch source for rapids-cmake in both the git-based fetch branch and the archive URL fetch branch.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: adding logging in RAPIDS.cmake for the rapids-cmake source.
Description check ✅ Passed The description references the issue tied to this change, so it is related to the pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants