Skip to content

[pre-commit.ci] pre-commit autoupdate #6138

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[pre-commit.ci] pre-commit autoupdate #6138

wants to merge 2 commits into from

Conversation

pre-commit-ci[bot]
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Oct 6, 2025

@pre-commit-ci
[pre-commit.ci] pre-commit autoupdate
fb17b70 
updates:
- [github.com/pre-commit/pre-commit-hooks: v5.0.0 → v6.0.0](pre-commit/pre-commit-hooks@v5.0.0...v6.0.0)
- [github.com/pre-commit/mirrors-clang-format: v20.1.7 → v21.1.2](pre-commit/mirrors-clang-format@v20.1.7...v21.1.2)
- [github.com/astral-sh/ruff-pre-commit: v0.12.2 → v0.13.3](astral-sh/ruff-pre-commit@v0.12.2...v0.13.3)
- [github.com/pre-commit/mirrors-mypy: v1.16.1 → v1.18.2](pre-commit/mirrors-mypy@v1.16.1...v1.18.2)
@pre-commit-ci pre-commit-ci bot requested a review from a team as a code owner October 6, 2025 21:59
@pre-commit-ci pre-commit-ci bot requested a review from jrhemstad October 6, 2025 21:59
@github-project-automation github-project-automation bot moved this to Todo in CCCL Oct 6, 2025
@copy-pr-bot copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Oct 6, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@cccl-authenticator-app cccl-authenticator-app bot moved this from Todo to In Review in CCCL Oct 6, 2025
@pre-commit-ci pre-commit-ci bot requested review from a team as code owners October 6, 2025 22:37
@pre-commit-ci pre-commit-ci bot requested a review from gonidelis October 6, 2025 22:37
gonidelis
gonidelis approved these changes Oct 6, 2025
View reviewed changes
Copy link
Member

@gonidelis gonidelis left a comment

Choose a reason for hiding this comment

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

Didn't know precommit can submit their own PRs. Not sure if this is triggered automatically or explicitly from one of us. I see some nice changes in formatting nevertheless. I guess approve?

@bernhardmgruber
Copy link
Contributor

/ok to test 1ce7b30

@bernhardmgruber
Copy link
Contributor

@alliepiper does this need to be synced with any devcontainers? Because of the clang-format version?

@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Oct 7, 2025

🥳 CI Workflow Results

🟩 Finished in 5h 44m: Pass: 100%/242 | Total: 11d 02h | Max: 5h 20m | Hits: 30%/340038

See results here.

@alliepiper
Copy link
Contributor

@alliepiper does this need to be synced with any devcontainers? Because of the clang-format version?

rapidsai/devcontainers#592

@alliepiper
Copy link
Contributor

Waiting on https://github.com/rapidsai/devcontainers/actions/runs/18321651522 to publish new images.

@alliepiper
Copy link
Contributor

Should be good to go now, devcontainers are updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bernhardmgruber bernhardmgruber bernhardmgruber approved these changes

@gonidelis gonidelis gonidelis approved these changes

@davebayer davebayer davebayer approved these changes

@jrhemstad jrhemstad Awaiting requested review from jrhemstad jrhemstad is a code owner automatically assigned from NVIDIA/cccl-infra-codeowners

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

CCCL
Status: In Review

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@bernhardmgruber @alliepiper @gonidelis @davebayer