Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 3 additions & 4 deletions .github/instructions/workflows.instructions.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,8 @@ When reviewing GitHub Actions workflows in this repository:
- Prefer simple, explicit Bash over clever one-liners when reliability is at stake.
- For new sanity or guard workflows, verify that initial branch pushes, pull_request events, and shallow clones are handled intentionally.
- Suggest minimal fixes rather than broad workflow rewrites unless the current approach is likely to fail.
- For local workflow-review parsing checks in the repo-managed environment, run `bash .codex/install.sh` first so the expected tooling is available.
- Before local workflow or tooling checks in the repo-managed environment, run `bash .codex/install.sh` so the expected tooling is available.
- Do not recommend `bash -n` against `.github/workflows/*.yml`; Bash syntax checking is not applicable to YAML workflow files.
- If shell validation is still needed, limit `bash -n` or `shellcheck` to real `.sh` files or extracted shell snippets.
- If review notes include a Python YAML parse example such as `python3 - <<'PY'` with `import yaml`, call out that it depends on PyYAML being installed.
- Prefer a guarded check like `python3 -c 'import yaml'` before YAML parsing when you want to separate dependency/setup issues from workflow content validation.
- Be explicit in review comments that a missing YAML parser dependency is a local tooling problem, not a GitHub Actions workflow syntax defect.
- Use an appropriate YAML-aware validator for workflow files.
- If a local YAML validator dependency is missing, report it as an environment or tooling limitation rather than a workflow syntax failure.
Loading