[codex] Apply audit remediation#88
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8f534d6de1
ℹ️ 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".
| monthly-publish: | ||
| if: github.ref_name != 'logs' | ||
| runs-on: | ||
| timeout-minutes: 60 |
There was a problem hiding this comment.
Restore the runner list under runs-on
In the monthly publish workflow, adding timeout-minutes at the same indentation as the empty runs-on: changes the YAML meaning: the runner labels are no longer assigned to runs-on and instead get folded into the timeout-minutes value (locally parsing this file yields runs-on: null and timeout-minutes: "60 - self-hosted - Linux - X64"). That makes the scheduled/manual monthly publish job invalid because GitHub Actions has no runner target and the timeout is not a number. Move timeout-minutes: 60 outside the runs-on block while keeping the self-hosted/Linux/X64 list under runs-on.
Useful? React with 👍 / 👎.
Summary
Validation
git diff --checkpassed across the remediation scopeSee local audit report:
QuantStrategyLab-review-2026-06-10.md.