Skip to content

fix(github): fix commit in "pull_request_target" event#274

Merged
gregberge merged 1 commit intomainfrom
fix-commit-pull_request_target
Feb 20, 2026
Merged

fix(github): fix commit in "pull_request_target" event#274
gregberge merged 1 commit intomainfrom
fix-commit-pull_request_target

Conversation

@gregberge
Copy link
Member

No description provided.

@gregberge gregberge requested review from a team and Copilot February 20, 2026 15:06
@vercel
Copy link

vercel bot commented Feb 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
argos-js-sdk-reference Ready Ready Preview, Comment Feb 20, 2026 4:37pm

Request Review

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request fixes an issue with commit SHA detection in GitHub Actions when using the pull_request_target event. In this event type, the GITHUB_SHA environment variable points to the base branch instead of the head commit of the pull request. The fix adds logic to detect this event and use the pull request's head SHA instead.

Changes:

  • Added special handling for pull_request_target events in the getSha function to use the PR head commit instead of the base branch commit
  • Added a new payload parameter to the getSha function to enable access to pull request information

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

@gregberge gregberge force-pushed the fix-commit-pull_request_target branch from 247fb80 to e8aae05 Compare February 20, 2026 16:36
@gregberge gregberge merged commit da07b9e into main Feb 20, 2026
65 checks passed
@gregberge gregberge deleted the fix-commit-pull_request_target branch February 20, 2026 16:41
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