Skip to content

Freeze pre-commit repo references and update secrets baseline - #759

Merged
jpodivin merged 2 commits into
packit:mainfrom
jpodivin:pre-commit-freeze
Aug 18, 2026
Merged

Freeze pre-commit repo references and update secrets baseline#759
jpodivin merged 2 commits into
packit:mainfrom
jpodivin:pre-commit-freeze

Conversation

@jpodivin

Copy link
Copy Markdown
Collaborator

By pinning repos to hashes instead of tag we prevent potential supply chain attack on developer machines. This also counts as an upgrade of the ruff pre commit.

Since the hashes do look somewhat like secrets, they have to be included in the baseline. New .secrects.baseline was generated with detect-secrets version 1.5.0, which is the same one as used by the pre-commit.

@jpodivin

Copy link
Copy Markdown
Collaborator Author

Pre-commit is failing due to non-compliant union. This should be fixed by: #758

@jpodivin
jpodivin force-pushed the pre-commit-freeze branch 2 times, most recently from 4657eae to 60a946d Compare August 18, 2026 12:40

@majamassarini majamassarini left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@jpodivin
jpodivin merged commit 5fad8b9 into packit:main Aug 18, 2026
11 checks passed
@jpodivin
jpodivin deleted the pre-commit-freeze branch August 18, 2026 14:16
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