Skip to content

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

Merged
rickeylev merged 1 commit into
bazel-contrib:mainfrom
rickeylev:update_workflow_rules
Aug 14, 2026
Merged

agents(rules): update github_actions_workflows rule with python script and testing conventions#4047
rickeylev merged 1 commit into
bazel-contrib:mainfrom
rickeylev:update_workflow_rules

Conversation

@rickeylev

Copy link
Copy Markdown
Collaborator

Workflow automation scripts and tests across GitHub Actions lacked
clear, standardized rules for console logging, test placement, Bazel
import targets, and test fixture design. Without shared guidelines,
scripts risked inconsistent log formatting, sys.path manipulations, and
ad-hoc test structures.

Update .agents/rules/github_actions_workflows.md to require GitHub
workflow command syntax for console logging, define tests under
tests/workflows/, prefer py_library imports over sys.path alterations,
and follow functional script design with autouse fixtures for mocks.

…t and testing conventions

Workflow automation scripts and tests across GitHub Actions lacked
clear, standardized rules for console logging, test placement, Bazel
import targets, and test fixture design. Without shared guidelines,
scripts risked inconsistent log formatting, sys.path manipulations, and
ad-hoc test structures.

Update .agents/rules/github_actions_workflows.md to require GitHub
workflow command syntax for console logging, define tests under
tests/workflows/, prefer py_library imports over sys.path alterations,
and follow functional script design with autouse fixtures for mocks.
@rickeylev
rickeylev marked this pull request as ready for review August 14, 2026 18:37
@rickeylev
rickeylev requested a review from aignas as a code owner August 14, 2026 18:37
@rickeylev
rickeylev merged commit 6009acb into bazel-contrib:main Aug 14, 2026
3 checks passed
@rickeylev
rickeylev deleted the update_workflow_rules branch August 14, 2026 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant