Skip to content
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

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

Merged
merged 1 commit into from
Dec 16, 2023

Conversation

pre-commit-ci[bot]
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Sep 12, 2023

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch 2 times, most recently from 8be5ae0 to 51c5003 Compare October 9, 2023 19:28
@Toreno96
Copy link
Collaborator

I went through the changelog of black from 23.7.0 to 23.10.1 and it LGTM. The remaining hooks are on my TODO list.

@Toreno96
Copy link
Collaborator

Toreno96 commented Dec 16, 2023

I've verified the changelogs of all those hooks, all LGTM.

I would consider bumping black to 23.12.0, as I already checked that one too.

In a separate PR, I would also consider replacing the current black hook with the officially recommended mirror which is about 2x faster:

repos:
  # Using this mirror lets us use mypyc-compiled black, which is about 2x faster
  - repo: https://github.com/psf/black-pre-commit-mirror

https://black.readthedocs.io/en/stable/integrations/source_version_control.html#version-control-integration

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.4.0 → v4.5.0](pre-commit/pre-commit-hooks@v4.4.0...v4.5.0)
- [github.com/psf/black: 23.7.0 → 23.12.0](psf/black@23.7.0...23.12.0)
- [github.com/asottile/reorder-python-imports: v3.10.0 → v3.12.0](asottile/reorder-python-imports@v3.10.0...v3.12.0)
@Toreno96 Toreno96 merged commit 7a72b8b into master Dec 16, 2023
2 checks passed
@Toreno96 Toreno96 deleted the pre-commit-ci-update-config branch December 16, 2023 14:34
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.

1 participant