Skip to content

Conversation

bolinfest
Copy link
Collaborator

@bolinfest bolinfest commented Sep 28, 2025

Two fixes:

  • changes the trigger from pull_request: { branches: [main] } to pull_request: {} (which is what we do in rust-ci.yml, for example) because if you require the base branch to be main and you use stacked PRs, the CI job only gets run for the PR on the bottom of the stack
  • prefer step outputs to env vars to pass information along

I started looking at the second issue because I thought the step wasn't working (though it is). The issue is that the PR UI here on GitHub will link you to the job:

https://github.com/openai/codex/actions/runs/18078793569/job/51439094961?pr=4408

and if you dig through the logs, you can get a download URL for the artifact, but the easier thing is to cut off the end of the URL and view this instead:

https://github.com/openai/codex/actions/runs/18078793569/

where the download link is prominent:

Screenshot 2025-09-28 at 12 35 51 PM

@bolinfest bolinfest merged commit 79e51dd into main Sep 28, 2025
19 checks passed
@bolinfest bolinfest deleted the pr4408 branch September 28, 2025 22:31
@github-actions github-actions bot locked and limited conversation to collaborators Sep 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants