Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 90deccca1d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| if: ${{ github.event_name != 'push' }} | ||
| uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4 | ||
| with: | ||
| ref: ${{ github.ref }} |
There was a problem hiding this comment.
For pull_request events this job checks out ${{ github.ref }} and then reads .github/ci/demo-build-config.json from that checkout, so the PR being validated can modify its own matrix/runner/ignored-error policy before jobs are generated. In practice, a PR can weaken required validation (for example by making entries informational or changing runner labels), which breaks CI’s role as an independent gate. Use a trusted ref for config on PRs (e.g., base SHA) or enforce a strict whitelist in-workflow.
Useful? React with 👍 / 👎.
…elist, documented manual run flow
|
Codex Review: Didn't find any major issues. Nice work! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
… Xcode, updated CI docs
2 participants
No description provided.