Skip to content

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

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
Merged
merged 5 commits into from
Jul 9, 2025
Merged

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

merged 5 commits into from
Jul 9, 2025

Conversation

pre-commit-ci[bot]
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Jul 7, 2025

@pre-commit-ci
[pre-commit.ci] pre-commit autoupdate
fa9019d 
updates:
- [github.com/astral-sh/ruff-pre-commit: 971923581912ef60a6b70dbf0c3e9a39563c9d47 → v0.12.2](astral-sh/ruff-pre-commit@9719235...v0.12.2)
- [github.com/PyCQA/bandit: 8ff25e07e487f143571cc305e56dd0253c60bc7b → 1.8.6](PyCQA/bandit@8ff25e0...1.8.6)
@github-project-automation github-project-automation bot moved this to Todo in CCCL Jul 7, 2025
@copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Jul 7, 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.

@kkraus14
Copy link
Collaborator

kkraus14 commented Jul 7, 2025 •
edited
Loading

Need to use --freeze to get SHAs instead of tags here. https://pre-commit.com/#pre-commit-autoupdate

pre-commit/pre-commit#3488

@kkraus14
Copy link
Collaborator

kkraus14 commented Jul 7, 2025

/ok to test

@copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Jul 7, 2025

/ok to test

@kkraus14, there was an error processing your request: E1

See the following link for more information: https://docs.gha-runners.nvidia.com/cpr/e/1/

@kkraus14
Copy link
Collaborator

kkraus14 commented Jul 7, 2025

/ok to test 7220141

asottile
asottile reviewed Jul 7, 2025
View reviewed changes
@github-actions

This comment has been minimized.

Sign in to view
@kkraus14
Copy link
Collaborator

kkraus14 commented Jul 7, 2025

/ok to test af0036e

@kkraus14
Copy link
Collaborator

kkraus14 commented Jul 7, 2025

/ok to test 875e955

@kkraus14
Copy link
Collaborator

kkraus14 commented Jul 7, 2025

@rwgk @leofang this could use a review due to the scope of changes from Ruff

rwgk
rwgk reviewed Jul 7, 2025
View reviewed changes
@leofang leofang added P1 Medium priority - Should do CI/CD CI/CD infrastructure cuda.core Everything related to the cuda.core module labels Jul 8, 2025
@leofang leofang added this to the cuda.core parking lot milestone Jul 8, 2025
@github-project-automation github-project-automation bot moved this from Todo to In Review in CCCL Jul 8, 2025
rwgk
rwgk approved these changes Jul 8, 2025
View reviewed changes
Copy link
Collaborator

@rwgk rwgk left a comment

Choose a reason for hiding this comment

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

I'm good with adopting the 3.10 syntax, too.

BTW: It even works with Python 3.8, with the __future__ import.

@kkraus14 kkraus14 merged commit 52874c1 into main Jul 9, 2025
1 check passed
@kkraus14 kkraus14 deleted the pre-commit-ci-update-config branch July 9, 2025 00:13
@github-project-automation github-project-automation bot moved this from In Review to Done in CCCL Jul 9, 2025
@github-actions
Copy link

github-actions bot commented Jul 9, 2025

Doc Preview CI
Preview removed because the pull request was closed or merged.

@kkraus14 kkraus14 mentioned this pull request Sep 25, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kkraus14 kkraus14 kkraus14 left review comments

@rwgk rwgk rwgk approved these changes

@leofang leofang leofang approved these changes

+1 more reviewer

@asottile asottile asottile left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@kkraus14 kkraus14

Labels

CI/CD CI/CD infrastructure cuda.core Everything related to the cuda.core module P1 Medium priority - Should do

Projects

CCCL
Archived in project

Milestone

cuda.core parking lot

Development

Successfully merging this pull request may close these issues.

4 participants

@kkraus14 @rwgk @asottile @leofang