Skip to content

[GH-3011] Pin all pre-commit hooks to SHA for supply chain security#3082

Open
jbampton wants to merge 1 commit into
apache:masterfrom
jbampton:pin-all-pre-commit-hooks
Open

[GH-3011] Pin all pre-commit hooks to SHA for supply chain security#3082
jbampton wants to merge 1 commit into
apache:masterfrom
jbampton:pin-all-pre-commit-hooks

Conversation

@jbampton

Copy link
Copy Markdown
Member

Did you read the Contributor Guide?

Is this PR related to a ticket?

What changes were proposed in this PR?

Pinned all hooks to latest SHA for supply chain security.

Using prek now so set minimum_prek_version similar to Apache Airflow:

https://github.com/apache/airflow/blob/b788d83d215ec9fe6b289c3021a726997ca3d8a3/.pre-commit-config.yaml#L19

Updated additional dependency version for Prettier

How was this patch tested?

With prek

Did this PR include necessary documentation updates?

  • No, this PR does not affect any public API so no need to change the documentation.

@jbampton jbampton added this to the sedona-1.9.1 milestone Jun 24, 2026
@jbampton jbampton added the pre-commit Pull requests that update pre_commit code label Jun 24, 2026
@jbampton jbampton requested a review from jiayuasu as a code owner June 24, 2026 06:58
@jiayuasu jiayuasu requested a review from Copilot June 25, 2026 16:11

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR strengthens the repository’s pre-commit/prek supply-chain posture by pinning hook revisions to immutable commit SHAs (with # frozen: hints for Dependabot), while also updating the configured minimum prek version and a Prettier dependency.

Changes:

  • Converted all third-party hook rev values from tags to full commit SHAs and added # frozen: version hints for Dependabot.
  • Switched the config’s minimum version gate from minimum_pre_commit_version to minimum_prek_version.
  • Bumped the local Prettier hook dependency from prettier@3.8.3 to prettier@3.8.4.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .pre-commit-config.yaml
python: python3
node: 24.16.0
minimum_pre_commit_version: '3.2.0'
minimum_prek_version: '0.3.4'
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pre-commit Pull requests that update pre_commit code security

Projects

None yet

Development

Successfully merging this pull request may close these issues.

We need to pin to sha all pre-commit hooks for supply chain security

2 participants