Skip to content

agents(rules): update github_actions_workflows rule with python script and testing conventions #4047

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
rickeylev merged 1 commit into from
Aug 14, 2026
+11 −0
Merged

agents(rules): update github_actions_workflows rule with python script and testing conventions #4047

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
11 changes: 11 additions & 0 deletions .agents/rules/github_actions_workflows.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,3 +10,14 @@ globs: [".github/workflows/*.yml", ".github/workflows/*.yaml", ".github/*.yaml"]
* Preserve `suppress-no-jobs-ran-error` fallback jobs in conditional workflows.
* Pass inputs (e.g. `${{ inputs.issue }}`) directly to commands without
redundant shell parameter stripping or conversions.
* Print console messages using GitHub workflow command syntax (e.g.,
`::error::`, `::warning::`, `::notice::`, `::group::`).

## Workflow Python Scripts & Testing
* **Test Location**: Place workflow tests under `tests/workflows/`.
* **Imports over `sys.path`**: Never alter `sys.path`. Define a `py_library`
with `imports = ["../../.github/workflows"]` in `tests/workflows/BUILD.bazel`.
* **Functional Design & Fixtures**: Keep scripts functional with one public
entry point (e.g. `process_comment()`) and private helpers (`_` prefix). Write
outputs directly when matching actions. Test end-to-end via autouse fixtures
for GHA env files and API mocks.