Problem
The header comment of .github/workflows/test.yml reads:
# Runs the test suite on every push and pull request so regressions are
# caught before merge, rather than relying solely on manual local validation.
The triggers immediately below it are narrower:
on:
push:
branches: [main]
pull_request:
A push to a feature branch is therefore only covered once a pull request has been opened for it. A branch pushed without one runs nothing, and the silence is indistinguishable from a green run.
The identical wording had also drifted into README.md, where it was corrected in PR #38. The workflow comment was deliberately left untouched there, because workflow files are excluded from autonomous change — this issue exists so the two do not stay out of step.
Suggested fix
Reword the comment to match the triggers, for example:
# Runs the test suite on every pull request and on every push to main, so
# regressions are caught before merge rather than relying solely on manual
# local validation. A branch pushed without an open pull request is not
# covered.
Broadening the trigger to all pushes is the alternative, at the cost of running the 3.9-3.13 matrix twice for every commit on a branch that already has a pull request open. Which of the two is wanted is a maintainer call; only the comment is wrong today.
This issue was drafted during a Gardener session (https://github.com/Stephenson-Software/gardener).
drafted by Claude on behalf of Daniel Stephenson
Problem
The header comment of
.github/workflows/test.ymlreads:The triggers immediately below it are narrower:
A push to a feature branch is therefore only covered once a pull request has been opened for it. A branch pushed without one runs nothing, and the silence is indistinguishable from a green run.
The identical wording had also drifted into
README.md, where it was corrected in PR #38. The workflow comment was deliberately left untouched there, because workflow files are excluded from autonomous change — this issue exists so the two do not stay out of step.Suggested fix
Reword the comment to match the triggers, for example:
Broadening the trigger to all pushes is the alternative, at the cost of running the 3.9-3.13 matrix twice for every commit on a branch that already has a pull request open. Which of the two is wanted is a maintainer call; only the comment is wrong today.
This issue was drafted during a Gardener session (https://github.com/Stephenson-Software/gardener).
drafted by Claude on behalf of Daniel Stephenson