Skip to content

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

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

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

Conversation

@arhag23

@arhag23 arhag23 commented Jul 6, 2026

Copy link
Copy Markdown

@arhag23 arhag23 force-pushed the rapids-cmake-log-source branch from 24f11bc to 33c658d Compare July 6, 2026 20:49
@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: 87bb7faa-fe1c-4326-906b-4edd66cee850

📥 Commits

Reviewing files that changed from the base of the PR and between 60c3e94 and 33c658d.

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

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated copyright text to include NVIDIA CORPORATION & AFFILIATES.
    • Improved build status messages to show clearer source and version information when fetching dependencies.

Walkthrough

This PR updates the SPDX copyright header in cmake/RAPIDS.cmake to include "& AFFILIATES" and modifies FetchContent status messaging to print the resolved repository URL with tag/SHA when fetching via git, and the resolved URL when not fetching via git.

Changes

RAPIDS.cmake update

Layer / File(s) Summary
Copyright header and FetchContent status messages
cmake/RAPIDS.cmake
SPDX copyright line updated to add "& AFFILIATES"; FetchContent status output updated to log repo URL with tag/SHA for git fetch path and resolved URL for non-git fetch path.

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

Related PRs: None identified.

Suggested labels: None

Suggested reviewers: None

🐰 A tiny tweak, a copyright line grows,
NVIDIA's name now with affiliates shows,
A URL logged with its tag in tow,
Cmake whispers softly where fetches go.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: adding logging for rapids-cmake source in RAPIDS.cmake.
Description check ✅ Passed The description is related to the change because it links the relevant issue and matches the stated logging update.
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.

@divyegala divyegala added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jul 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

3 participants