Skip to content

ci: isolate smoke-asset download and fix PR code-quality checkout - #924

Merged
kirithika7 merged 1 commit into
Multicorewareinc:masterfrom
Akilan-Sivakumar:master
Jul 24, 2026
Merged

ci: isolate smoke-asset download and fix PR code-quality checkout#924
kirithika7 merged 1 commit into
Multicorewareinc:masterfrom
Akilan-Sivakumar:master

Conversation

@Akilan-Sivakumar

Copy link
Copy Markdown
Collaborator
  • Fixes fork-PR CI failure: Refusing to check out fork pull request code from a 'pull_request_target' workflow.
  • code-quality: adds allow-unsafe-pr-checkout: true to opt in (this job only reads changed files - it never builds/runs PR code) and persist-credentials: false so the token isn't left on disk.
  • Adds a prep-assets job that downloads the smoke tarballs with the asset token and republishes them as an internal smoke-assets artifact - it checks out no PR code, so the token can't be exfiltrated.
  • smoke-linux and smoke-windows now consume that artifact instead of holding the token, keeping the secret out of every job that runs fork code.

@kirithika7
kirithika7 merged commit 8a55d60 into Multicorewareinc:master Jul 24, 2026
14 of 16 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