Skip to content

Update to clang 20.1.8#1090

Merged
rapids-bot[bot] merged 1 commit intoNVIDIA:mainfrom
bdice:clang-20.1.8
Apr 10, 2026
Merged

Update to clang 20.1.8#1090
rapids-bot[bot] merged 1 commit intoNVIDIA:mainfrom
bdice:clang-20.1.8

Conversation

@bdice
Copy link
Copy Markdown
Contributor

@bdice bdice commented Apr 10, 2026

Description

Contributes to rapidsai/build-planning#267

This PR updates the clang version to 20.1.8.

@bdice bdice added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Apr 10, 2026
@bdice bdice requested a review from a team as a code owner April 10, 2026 02:23
@bdice bdice requested a review from msarahan April 10, 2026 02:23
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 10, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: aa79d45f-7fa6-4b8b-8fb4-1dcb70f7ea1c

📥 Commits

Reviewing files that changed from the base of the PR and between f5842df and e92ca1d.

📒 Files selected for processing (1)
  • .pre-commit-config.yaml

📝 Walkthrough

Walkthrough

Updated the clang-format pre-commit hook version from v20.1.4 to v20.1.8 in the pre-commit configuration file. This is a patch version update to the formatter used for C/C++ source and header files.

Changes

Cohort / File(s) Summary
Pre-commit Configuration
.pre-commit-config.yaml
Bumped mirrors-clang-format hook version from v20.1.4 to v20.1.8.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Update to clang 20.1.8' directly and clearly describes the main change in the pull request, which updates the clang-format version from v20.1.4 to v20.1.8.
Description check ✅ Passed The description is directly related to the changeset, explaining that it updates the clang version to 20.1.8 and references the associated issue.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@jameslamb jameslamb removed the request for review from msarahan April 10, 2026 14:35
@bdice
Copy link
Copy Markdown
Contributor Author

bdice commented Apr 10, 2026

/merge

@rapids-bot rapids-bot bot merged commit 24fdb60 into NVIDIA:main Apr 10, 2026
76 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