Skip to content

ci: fix fork-PR checkout blocked by security guard - #172

Merged
YevhS merged 3 commits into
mate-academy:mainfrom
viktoriia-boiko:pull-request-target-checkout-security-fix
Jul 27, 2026
Merged

ci: fix fork-PR checkout blocked by security guard#172
YevhS merged 3 commits into
mate-academy:mainfrom
viktoriia-boiko:pull-request-target-checkout-security-fix

Conversation

@viktoriia-boiko

Copy link
Copy Markdown
Contributor

GitHub added a security guard to actions/checkout that refuses to check out fork PR code inside a pull_request_target workflow (the pattern this repo used to build student PRs and then auto-approve/auto-reject). This currently blocks CI on every student pull request.

This splits the workflow the same way it was already fixed for the py-* task repos on 2026-07-21:

  • test.yml: untrusted, triggers on plain pull_request, builds the fork's code (no secrets), uploads the PR number as an artifact.
  • pr-review.yml: trusted, triggers on workflow_run after Test completes, never checks out fork code, reads the PR number artifact and runs auto-approve-action/auto-reject-action.

Ref: RQ-13187

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Some actions are not successful, please fix it.

@YevhS
YevhS merged commit 99edc4f into mate-academy:main Jul 27, 2026
0 of 2 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