Skip to content

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

Open
pre-commit-ci[bot] wants to merge 4 commits into
masterfrom
pre-commit-ci-update-config
Open

[pre-commit.ci] pre-commit autoupdate#518
pre-commit-ci[bot] wants to merge 4 commits into
masterfrom
pre-commit-ci-update-config

Conversation

@pre-commit-ci

@pre-commit-ci pre-commit-ci Bot commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

pre-commit-ci Bot added 2 commits April 6, 2026 19:36
updates:
- [github.com/pre-commit/mirrors-clang-format: v21.1.8 → v22.1.2](pre-commit/mirrors-clang-format@v21.1.8...v22.1.2)
- [github.com/psf/black-pre-commit-mirror: 25.12.0 → 26.3.1](psf/black-pre-commit-mirror@25.12.0...26.3.1)
@arntanguy

Copy link
Copy Markdown
Collaborator

Not a huge fan of the one-line function changes here, I might want to update the .clang-format configuration before merging

@ThomasDuvinage

Copy link
Copy Markdown
Contributor

@arntanguy
Probably not the cleanest way to do it even thought it fixes the issue.

@arntanguy

arntanguy commented Apr 8, 2026

Copy link
Copy Markdown
Collaborator

Not ideal as it changes a lot of other one-liner that were originally set-up on purpose by Pierre (we might want to revisit this down the line, I think I like the proposed more verbose style better). This will probably cause some rebasing headaches if we merge it as-is. I'll try to have a look, nothing urgent, we can leave pre-commit as-is in the meanwhile.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants