Skip to content

Update pre-commit-hooks, fix UCXX version scheme#3087

Merged
rapids-bot[bot] merged 2 commits into
NVIDIA:release/26.08from
KyleFromNVIDIA:fix-pre-commit-hooks
Jul 20, 2026
Merged

Update pre-commit-hooks, fix UCXX version scheme#3087
rapids-bot[bot] merged 2 commits into
NVIDIA:release/26.08from
KyleFromNVIDIA:fix-pre-commit-hooks

Conversation

@KyleFromNVIDIA

Copy link
Copy Markdown
Member

Update pre-commit-hooks to get a bugfix related to the --version-scheme argument, and use this argument in verify-hardcoded-version-ucxx.

@KyleFromNVIDIA KyleFromNVIDIA added the bug Something isn't working label Jul 20, 2026
@KyleFromNVIDIA
KyleFromNVIDIA requested a review from a team as a code owner July 20, 2026 18:26
@KyleFromNVIDIA KyleFromNVIDIA added the non-breaking Non-breaking change label Jul 20, 2026
@KyleFromNVIDIA
KyleFromNVIDIA requested a review from bdice July 20, 2026 18:26
@coderabbitai

coderabbitai Bot commented Jul 20, 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: d2fc368a-caed-49db-9eda-3ea9bf893a30

📥 Commits

Reviewing files that changed from the base of the PR and between c3e13c6 and e5e0a27.

📒 Files selected for processing (1)
  • cpp/tests/stats/mean.cu

📝 Walkthrough

Summary by CodeRabbit

  • Chores

    • Updated development validation tooling to the latest patch version.
    • Improved semantic-version validation for hardcoded version checks.
  • Tests

    • Reformatted mean test input data without changing test values or expected behavior.

Walkthrough

The pre-commit hooks revision was updated from v1.6.0 to v1.6.1, the UCXX verification hook gained semver handling, and the affected mean test initializer was reformatted with a pre-commit suppression comment.

Changes

Pre-commit configuration and test formatting

Layer / File(s) Summary
Update hook revision and validation arguments
.pre-commit-config.yaml
The hooks pin advances to v1.6.1, and verify-hardcoded-version-ucxx adds --version-scheme=semver while retaining its existing arguments.
Adjust affected mean test initializer
cpp/tests/stats/mean.cu
The inputsi8h initializer is reformatted across multiple lines without changing numeric values, and a pre-commit disable-next-line comment is added.

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

Suggested reviewers: bdice

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the main change: updating pre-commit-hooks and fixing UCXX version-scheme handling.
Description check ✅ Passed The description directly describes the same change set: a pre-commit-hooks update and using --version-scheme in verify-hardcoded-version-ucxx.
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.

@KyleFromNVIDIA
KyleFromNVIDIA changed the base branch from main to release/26.08 July 20, 2026 18:31
@KyleFromNVIDIA
KyleFromNVIDIA requested a review from a team as a code owner July 20, 2026 18:32
@KyleFromNVIDIA

Copy link
Copy Markdown
Member Author

/merge

@rapids-bot
rapids-bot Bot merged commit bbca7b1 into NVIDIA:release/26.08 Jul 20, 2026
80 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants