Skip to content

Attempt to find cmake-format-rapids-cmake.json in cpp/latest#7626

Merged
rapids-bot[bot] merged 3 commits intorapidsai:mainfrom
divyegala:cmake-format-paths
Dec 18, 2025
Merged

Attempt to find cmake-format-rapids-cmake.json in cpp/latest#7626
rapids-bot[bot] merged 3 commits intorapidsai:mainfrom
divyegala:cmake-format-paths

Conversation

@divyegala
Copy link
Copy Markdown
Member

This always is an issue otherwise when committing from devcontainers.

@divyegala divyegala self-assigned this Dec 17, 2025
@divyegala divyegala requested a review from a team as a code owner December 17, 2025 23:22
@divyegala divyegala added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Dec 17, 2025
Copy link
Copy Markdown
Contributor

@bdice bdice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's match cudf's logic. This should work for cuML too: https://github.com/rapidsai/cudf/blob/main/cpp/scripts/run-cmake-format.sh#L46-L50

Maybe just copy-paste the latest version of that script into cuML and replace s/cudf/cuml/g.

@divyegala divyegala changed the title Attempt to find cmake-format-rapids-cmake.json if not found Attempt to find cmake-format-rapids-cmake.json in cpp/latest Dec 17, 2025
@divyegala
Copy link
Copy Markdown
Member Author

/merge

@rapids-bot rapids-bot Bot merged commit 5805c62 into rapidsai:main Dec 18, 2025
205 of 209 checks passed
mani-builds pushed a commit to mani-builds/cuml that referenced this pull request Jan 11, 2026
…idsai#7626)

This always is an issue otherwise when committing from devcontainers.

Authors:
  - Divye Gala (https://github.com/divyegala)

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

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

Labels

CUDA/C++ 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