Skip to content

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

Merged
jiayuasu merged 3 commits into
apache:masterfrom
jbampton:pin-all-pre-commit-hooks
Jul 14, 2026
Merged

[GH-3011] Pin all pre-commit hooks to SHA for supply chain security#3082
jiayuasu merged 3 commits 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 Outdated
@jiayuasu

jiayuasu commented Jul 6, 2026

Copy link
Copy Markdown
Member

@jbampton please address conflict

jbampton and others added 2 commits July 15, 2026 03:39
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@jbampton
jbampton requested a review from Copilot July 14, 2026 18:05

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@jiayuasu
jiayuasu merged commit e7c1224 into apache:master Jul 14, 2026
11 checks passed
@jbampton
jbampton deleted the pin-all-pre-commit-hooks branch August 2, 2026 11:06
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

3 participants