Skip to content

fix(hygiene action): checkout head ref on forked repos#1752

Closed
aditya-mitra wants to merge 3 commits intospeakeasy-api:mainfrom
aditya-mitra:hygiene-action
Closed

fix(hygiene action): checkout head ref on forked repos#1752
aditya-mitra wants to merge 3 commits intospeakeasy-api:mainfrom
aditya-mitra:hygiene-action

Conversation

@aditya-mitra
Copy link
Contributor

@aditya-mitra aditya-mitra commented Mar 2, 2026

git checkout on PR head ref on external repo instead of main repo in the "Lint PR Title and Changesets" github action

example of a run this PR fixes: https://github.com/speakeasy-api/gram/actions/runs/22520577348/job/65326453122?pr=1727


Open with Devin

@aditya-mitra aditya-mitra requested a review from a team as a code owner March 2, 2026 17:31
@vercel
Copy link

vercel bot commented Mar 2, 2026

@aditya-mitra is attempting to deploy a commit to the Speakeasy Team on Vercel.

A member of the Team first needs to authorize it.

@changeset-bot
Copy link

changeset-bot bot commented Mar 2, 2026

⚠️ No Changeset found

Latest commit: 3513415

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@aditya-mitra aditya-mitra changed the title Hygiene action fix(hygiene action): checkout head ref on forked repos Mar 2, 2026
@disintegrator
Copy link
Contributor

Will relax this check if a PR comes from a fork but we will not be automatically checking out forked repos in CI.

@github-actions github-actions bot locked and limited conversation to collaborators Mar 9, 2026
@aditya-mitra aditya-mitra deleted the hygiene-action branch March 9, 2026 14:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants