Skip to content

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

Merged
kkraus14 merged 5 commits into
mainfrom
pre-commit-ci-update-config
Jul 9, 2025
Merged

[pre-commit.ci] pre-commit autoupdate#746
kkraus14 merged 5 commits into
mainfrom
pre-commit-ci-update-config

Conversation

@pre-commit-ci

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

Copy link
Copy Markdown
Contributor

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)
@copy-pr-bot

copy-pr-bot Bot commented Jul 7, 2025

Copy link
Copy Markdown
Contributor

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

kkraus14 commented Jul 7, 2025

Copy link
Copy Markdown
Collaborator

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

pre-commit/pre-commit#3488

@kkraus14

kkraus14 commented Jul 7, 2025

Copy link
Copy Markdown
Collaborator

/ok to test

@copy-pr-bot

copy-pr-bot Bot commented Jul 7, 2025

Copy link
Copy Markdown
Contributor

/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

kkraus14 commented Jul 7, 2025

Copy link
Copy Markdown
Collaborator

/ok to test 7220141

Comment thread .pre-commit-config.yaml Outdated
@github-actions

This comment has been minimized.

@kkraus14

kkraus14 commented Jul 7, 2025

Copy link
Copy Markdown
Collaborator

/ok to test af0036e

@kkraus14

kkraus14 commented Jul 7, 2025

Copy link
Copy Markdown
Collaborator

/ok to test 875e955

@kkraus14

kkraus14 commented Jul 7, 2025

Copy link
Copy Markdown
Collaborator

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

Comment thread .pre-commit-config.yaml
Comment thread cuda_core/cuda/core/experimental/_graph.py
@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 left a comment

Copy link
Copy Markdown
Contributor

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
@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

github-actions Bot commented Jul 9, 2025

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

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

Labels

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

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants