Skip to content

ci: allow fork checkout in submission validator - #116

Closed
duongja wants to merge 1 commit into
logos-co:masterfrom
duongja:fix/allow-fork-pr-checkout
Closed

ci: allow fork checkout in submission validator#116
duongja wants to merge 1 commit into
logos-co:masterfrom
duongja:fix/allow-fork-pr-checkout

Conversation

@duongja

@duongja duongja commented Jul 30, 2026

Copy link
Copy Markdown

This fixes the Validate Submission workflow for fork-based solution PRs.

Current failure on LP-0002 submission PR #115:

The workflow already treats the PR-head checkout as untrusted data and does not execute code from it. It checks out the fork only so the trusted base-branch validation script can inspect submitted markdown content and changed files.

Change:

  • set allow-unsafe-pr-checkout: true on the PR-head actions/checkout@v4 step.

After this merges, fork-based solution PRs such as #115 can be rerun by the normal validator workflow.

@github-actions

Copy link
Copy Markdown

⚠️ Validation script did not produce output.

@duongja

duongja commented Jul 30, 2026

Copy link
Copy Markdown
Author

Closing this as a duplicate of #109.

#109 is the same fix for the same pull_request_target fork-checkout guard that is currently blocking fork-based solution PR validation, including LP-0002 #115.

@duongja duongja closed this Jul 30, 2026
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.

1 participant