Skip to content

refactor(release): move on_comment workflow logic into Python script - #4045

Merged
rickeylev merged 3 commits into
bazel-contrib:mainfrom
rickeylev:refactor_comment_script_logic
Aug 14, 2026
Merged

refactor(release): move on_comment workflow logic into Python script#4045
rickeylev merged 3 commits into
bazel-contrib:mainfrom
rickeylev:refactor_comment_script_logic

Conversation

@rickeylev

Copy link
Copy Markdown
Collaborator

The inline bash parsing logic in the comment dispatch workflow was
becoming complex and difficult to maintain or verify without executing
live GitHub Actions runs.

Move the comment dispatch logic into a dedicated Python script running
under Python 3.14, add a Bazel py_library target, and establish full
unit test coverage using pytest.

Move the comment dispatch logic from on_comment.yaml into a standalone
Python script (.github/workflows/on_comment.py) to improve testability
and maintainability. Add comprehensive pytest test coverage in
//tests/tools/private/release:on_comment_test.
@rickeylev
rickeylev requested a review from aignas as a code owner August 14, 2026 15:09
@rickeylev
rickeylev merged commit ed2c1bd into bazel-contrib:main Aug 14, 2026
3 of 4 checks passed
@rickeylev
rickeylev deleted the refactor_comment_script_logic branch August 14, 2026 18:24
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