Skip to content

Format all CMake code#821

Merged
rapids-bot[bot] merged 2 commits intorapidsai:branch-25.06from
bdice:format-all-cmake
Apr 10, 2025
Merged

Format all CMake code#821
rapids-bot[bot] merged 2 commits intorapidsai:branch-25.06from
bdice:format-all-cmake

Conversation

@bdice
Copy link
Copy Markdown
Contributor

@bdice bdice commented Apr 9, 2025

Description

This PR enables pre-commit rules to format all CMake code.

cmake-format now runs on the entire repository.

The testing/ directory is excluded from cmake-lint but all other code is linted.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.
  • The cmake-format.json is up to date with these changes.
  • I have added new files under rapids-cmake/
    • I have added include guards (include_guard(GLOBAL))
    • I have added the associated docs/ rst file and update the api.rst

@bdice bdice requested review from a team as code owners April 9, 2025 22:27
@bdice bdice requested review from msarahan and removed request for a team April 9, 2025 22:27
@bdice bdice added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Apr 9, 2025
@bdice bdice force-pushed the format-all-cmake branch from d0365dd to 421ea40 Compare April 9, 2025 22:30
@bdice bdice force-pushed the format-all-cmake branch from 421ea40 to 92d9f6f Compare April 9, 2025 22:50
@bdice
Copy link
Copy Markdown
Contributor Author

bdice commented Apr 10, 2025

/merge

@rapids-bot rapids-bot bot merged commit bcf2487 into rapidsai:branch-25.06 Apr 10, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants