-
Notifications
You must be signed in to change notification settings - Fork 128
[pre-commit.ci] pre-commit autoupdate #38314
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.
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exciting update
As this is a ruff change I am waiting for all of @robertapplin ruff related PRs to go in first! Hence why I had not reviewed it yet. |
#38319 is waiting for a gatekeeper if anyone has a spare moment :) |
I doubt that changing the patch level version of ruff will affect these files. A way to find out is to pull the commit into those PRs. |
0bbec2a
to
b239ef6
Compare
updates: - [github.com/pre-commit/mirrors-clang-format: v19.1.2 → v19.1.3](pre-commit/mirrors-clang-format@v19.1.2...v19.1.3) - [github.com/astral-sh/ruff-pre-commit: v0.7.0 → v0.7.2](astral-sh/ruff-pre-commit@v0.7.0...v0.7.2)
b239ef6
to
43d839e
Compare
@sf1919 @robertapplin and I agreed that #38345 can be merged after this. |
test failure is unrelated |
updates: