Skip to content

fix(workflow): use pull_request_target for PR triage#517

Merged
Harxhit merged 1 commit into
Dev-Card:mainfrom
Harxhit:triage-fix
Jun 9, 2026
Merged

fix(workflow): use pull_request_target for PR triage#517
Harxhit merged 1 commit into
Dev-Card:mainfrom
Harxhit:triage-fix

Conversation

@Harxhit

@Harxhit Harxhit commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

Switches the pull request triage workflow from pull_request to pull_request_target to ensure the workflow has the required permissions to apply labels and post comments on pull requests.

During testing, workflow logs showed that the GITHUB_TOKEN was being granted read-only permissions, causing label and comment operations to fail with 403 Resource not accessible by integration. Using pull_request_target resolves this by running the workflow with the permissions of the target repository.


What Changed

  • Updated the PR triage workflow trigger from pull_request to pull_request_target.
  • Fixed permission issues preventing automatic labeling and reviewer notification.
  • Verified the workflow can access the required write permissions for PR triage actions.

@vercel

vercel Bot commented Jun 9, 2026

Copy link
Copy Markdown

@Harxhit is attempting to deploy a commit to the Prashantkumar Khatri's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

CI — All Checks Passed

Backend — SKIP

Check Result
Lint -
Test -
Typecheck -

Mobile — SKIP

Check Result
Lint -
Test -

Web — SKIP

Check Result
Check -
Build -

Last updated: Tue, 09 Jun 2026 08:11:31 GMT

@Harxhit Harxhit requested a review from ShantKhatri June 9, 2026 08:11
@Harxhit

Harxhit commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator Author

@ShantKhatri Please review when you get a chance. I switched the workflow to pull_request_target to fix the permission issues we hit during testing.

@Harxhit Harxhit merged commit ad238cf into Dev-Card:main Jun 9, 2026
6 of 7 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