Skip to content

Switch lat check to pull_request_target so fork PRs run without maintainer approval.#60

Open
lars20070 wants to merge 6 commits into
1st1:mainfrom
lars20070:lars20070/bypassapproval
Open

Switch lat check to pull_request_target so fork PRs run without maintainer approval.#60
lars20070 wants to merge 6 commits into
1st1:mainfrom
lars20070:lars20070/bypassapproval

Conversation

@lars20070

@lars20070 lars20070 commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

Summary

Run the lat check CI workflow automatically on fork PRs. No maintainer approval required.

Why

Fork PRs fire pull_request, which parks the run behind the GitHub's "Approve and run" gate. Contributors cannot see in the CI whether lat check passes or not.

Changed

  • Switch the trigger from pull_request to pull_request_target. It isn't subject to the fork-PR approval gate, so the workflow now starts on its own.
  • Check out the PR's head SHA explicitly, since pull_request_target defaults to the base branch.
  • Keep previous permissions: contents: read. No PR code is executed. Skipping the gate for the lars20070/lat-check-action@v1 action is safe. Feel free to move to 1st1/lat-check-action@v1.
  • All instances of .DS_Store are ignored.

@vercel

vercel Bot commented Apr 21, 2026

Copy link
Copy Markdown

@lars20070 is attempting to deploy a commit to the Yury Selivanov's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

1 participant