Skip to content

Pinning pre-commit hooks to hashes#35

Merged
jpodivin merged 1 commit into
fedora-copr:mainfrom
jpodivin:pre-commit-freeze
Apr 20, 2026
Merged

Pinning pre-commit hooks to hashes#35
jpodivin merged 1 commit into
fedora-copr:mainfrom
jpodivin:pre-commit-freeze

Conversation

@jpodivin

Copy link
Copy Markdown
Contributor

This also updates hooks where applicable.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the .pre-commit-config.yaml file by pinning several hooks to specific commit hashes. The review feedback points out critical issues regarding version compatibility and accuracy: the uv version exceeds the constraints set in pyproject.toml, the pre-commit-hooks version label is misleading, and the ruff version label refers to a release that does not exist in the upstream repository.

Comment thread .pre-commit-config.yaml
Comment thread .pre-commit-config.yaml
Comment thread .pre-commit-config.yaml
Signed-off-by: Jiri Podivin <jpodivin@redhat.com>
@jpodivin jpodivin merged commit 9191f01 into fedora-copr:main Apr 20, 2026
3 checks passed
@jpodivin jpodivin deleted the pre-commit-freeze branch April 20, 2026 07:43
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