Skip to content

fix: run license check safely for fork PRs - #511

Merged
morningman merged 1 commit into
apache:masterfrom
Al-assad:codex/fix-license-check-fork-prs
Aug 4, 2026
Merged

fix: run license check safely for fork PRs#511
morningman merged 1 commit into
apache:masterfrom
Al-assad:codex/fix-license-check-fork-prs

Conversation

@Al-assad

@Al-assad Al-assad commented Aug 4, 2026

Copy link
Copy Markdown
Member

Summary

  • run the license workflow on pull_request instead of pull_request_target
  • upgrade checkout to actions/checkout@v4
  • use the default low-privilege fork PR checkout instead of opting into unsafe checkout

Validation

  • parsed .github/workflows/license-eyes.yml successfully as YAML
  • git diff --check

Context

The existing pull_request_target job now fails before the license scan because actions/checkout refuses to check out fork code in a privileged workflow. This PR itself may still show that legacy check failure because pull_request_target workflows are loaded from the default branch; the new workflow takes effect after merge.

@morningman
morningman merged commit 9b8952e into apache:master Aug 4, 2026
3 of 4 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