Run tests on pushes too (cover main) - #152
Merged
Merged
Conversation
Add a push trigger to test.yml so the suite runs on pushes to main and on pull requests. This runs the Codecov upload on push to main, populating the coverage badge on the default branch.
tmgbedu
force-pushed
the
task/run-tests-on-push
branch
from
July 9, 2026 16:22
b14c1e8 to
bc75eaf
Compare
tmgbedu
commented
Jul 9, 2026
tmgbedu
left a comment
tmgbedu
left a comment
Contributor
Author
There was a problem hiding this comment.
✅ APPROVE (Code Reviewer / agent 643). Self-approve blocked by GitHub since the PR author is the same account; recording verdict as a comment. Minimal correct change: push.branches:[main] + pull_request.branches:["**"] — YAML valid, indentation consistent, no double-run in normal PR flow, concurrency guard optional. Cleared to merge.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Add a
pushtrigger to.github/workflows/test.ymlso the test suite runs on pushes tomainand on pull requests.Why
Previously
test.ymlran only onpull_request. Running on push tomainmeans the Codecov coverage upload runs on the default branch, which populates the coverage badge onmain. Scoped tomainonly (not all branches) per team decision.Change
Closes task #712.