Skip to content

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

Merged
cclauss merged 1 commit into
developfrom
pre-commit-ci-update-config
Jun 2, 2026
Merged

[pre-commit.ci] pre-commit autoupdate#438
cclauss merged 1 commit into
developfrom
pre-commit-ci-update-config

Conversation

@pre-commit-ci

@pre-commit-ci pre-commit-ci Bot commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.15.12 → v0.15.15](astral-sh/ruff-pre-commit@v0.15.12...v0.15.15)
- [github.com/tox-dev/pyproject-fmt: v2.21.1 → v2.23.0](tox-dev/pyproject-fmt@v2.21.1...v2.23.0)
@aaronliu0130

Copy link
Copy Markdown
Member

I think it might be better to let the pre-commit updates accumulate here until right before we start working on a PR. When there's multiple, they get lumped into the same PR, and we only have to merge once.

@cclauss cclauss merged commit 0c644c0 into develop Jun 2, 2026
11 checks passed
@cclauss cclauss deleted the pre-commit-ci-update-config branch June 2, 2026 05:23
@cclauss

cclauss commented Jun 2, 2026

Copy link
Copy Markdown
Member

I disagree. These updates are scheduled monthly (instead of weekly), and the PRs ensure that at least one of us reviews the codebase and keeps it current with changes in C, C++, and Python, as well as the pre-commit tools.
That is completely separate from when we choose to make a new release to PyPI.

ci:
autoupdate_schedule: monthly

@aaronliu0130

aaronliu0130 commented Jun 2, 2026

Copy link
Copy Markdown
Member

the PRs ensure that at least one of us reviews the codebase and keeps it current with changes in C, C++, and Python

I don't think that is true. Nothing pre-commit does makes us support any more of C/C++, nor follow new releases of Python unless we change our policy for bumping the required Python version.

It's not making us make changes to update it so, either. The bugs about language support still stay there and such PRs seldom appear due to autoupdate PRs. Deciding to merge this PR has no advantage over setting a personal monthly timer. Plus, new changes being added to a PR monthly still generates the same notifications.

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