Skip to content

Add j178/prek-action to the GitHub Actions allowlist#782

Merged
potiuk merged 2 commits intoapache:mainfrom
potiuk:allowlist-add-j178-prek-action
May 2, 2026
Merged

Add j178/prek-action to the GitHub Actions allowlist#782
potiuk merged 2 commits intoapache:mainfrom
potiuk:allowlist-add-j178-prek-action

Conversation

@potiuk
Copy link
Copy Markdown
Member

@potiuk potiuk commented Apr 28, 2026

Add j178/prek-action to the allowlist:

  • j178/prek-action@cbc2f23eb5539cf20d82d1aabd0d0ecbcc56f4e3 (v2.0.2)

Needed by: apache/airflow-steward

Note

Originally proposed at commit 0bb87d7f (old composite-action era using curl https://github.com/j178/prek/releases/download/.../prek-installer.sh | sh with no in-action checksum step). Updated to v2.0.2 — the action was rewritten as a TypeScript JS action that uses @actions/tool-cache + crypto.createHash('sha256') to verify each downloaded artefact against checksums from the bundled known-checksums.ts manifest. verify-action-build passes cleanly on this commit.

@potiuk potiuk requested a review from dfoulks1 as a code owner April 28, 2026 23:59
@potiuk potiuk force-pushed the allowlist-add-j178-prek-action branch 4 times, most recently from 0503873 to 2d3c942 Compare April 29, 2026 00:06
@potiuk potiuk force-pushed the allowlist-add-j178-prek-action branch from 2d3c942 to f6c6e92 Compare April 29, 2026 00:08
@potiuk potiuk requested review from dave2wave and raboof April 29, 2026 00:09
Switch from the original commit (0bb87d7f, old composite-action era using
unverified curl-pipe-to-shell of prek-installer.sh) to v2.0.2
(cbc2f23e), the latest tagged release. v2.0.2 is a TypeScript JS action
that downloads release tarballs via @actions/tool-cache and verifies
each artefact with crypto.createHash('sha256') against checksums from
the action's bundled known-checksums.ts manifest.

verify-action-build now passes cleanly: binary download verification ✓,
JS build verification ✓ (compiled JS matches rebuild), all other
checks ✓.

Generated-by: Claude Opus 4.7 (1M context)
@potiuk
Copy link
Copy Markdown
Member Author

potiuk commented Apr 30, 2026

🙏

@potiuk
Copy link
Copy Markdown
Member Author

potiuk commented May 1, 2026

@ppkarwasz @dfoulks1 @dave2wave ?

@potiuk potiuk merged commit 5bba1f5 into apache:main May 2, 2026
10 checks passed
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