Skip to content

CI: fix pr-labeler permissions for external contributions - #87

Merged
Maximus7474 merged 2 commits into
mainfrom
ci/fix-labeler-perms
Sep 16, 2026
Merged

Maximus7474 merged 2 commits into
mainfrom
ci/fix-labeler-perms

Conversation

@Maximus7474

@Maximus7474 Maximus7474 commented Sep 16, 2026

Copy link
Copy Markdown
Member

Description

This PR fixes an issue encountered on #86 where the labeling workflow was unable to run its course and set the relevant labels on PRs from external forks. The changes made on this branch resolve the issue by:

  • Replacing the pull_request trigger with pull_request_target - allowing the workflow to run in the context of the base repository, which carries write permissions for PR labels.
  • Replacing actions/checkout and local git log commands in the commit inspection job with an API call via actions/github-script - fetching commit messages securely without checking out untrusted fork code.

Generative AI Disclosure

Note

Be honest—using GenAI does not penalize your PR, but failing to declare its use may result in the PR not being considered.

  • No, Generative AI was not used in the creation of this PR.
  • Yes, Generative AI was used in the creation of this PR.
    (If yes, please briefly describe what tools were used and for what purpose below):
    • Tools used: Gemini
    • Purpose / What it was used for: Trouble shooting the commit inspection job

Tests

  • N/A CI only

Additional Notes

@Maximus7474 Maximus7474 self-assigned this Sep 16, 2026
@Maximus7474
Maximus7474 marked this pull request as draft September 16, 2026 20:35
@Maximus7474
Maximus7474 marked this pull request as ready for review September 16, 2026 21:05
@Maximus7474
Maximus7474 merged commit 625f2d4 into main Sep 16, 2026
5 checks passed
@Maximus7474
Maximus7474 deleted the ci/fix-labeler-perms branch September 16, 2026 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant