Skip to content

Commit 137f34e

Browse files
eyalk11claude
andcommitted
ci(tests): always run tests on [newver] / [rel] commits
The trivial-push threshold was skipping the test job on version-bump commits, which are the worst time to skip — [newver] gates the build and [rel] gates the GitHub release, so a regression there ships straight to users. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent d6ee90f commit 137f34e

2 files changed

Lines changed: 13 additions & 1 deletion

File tree

.github/workflows/tests.yml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,18 @@ jobs:
3535
echo "workflow_dispatch -> always run"
3636
exit 0
3737
fi
38+
# Force tests on commits that gate the build chain: [newver]
39+
# triggers build-installer, [rel] triggers a GitHub release.
40+
# Skipping tests on those pushes is exactly the worst time to
41+
# skip them.
42+
msg='${{ github.event.head_commit.message }}'
43+
case "$msg" in
44+
*'[newver]'*|*'[rel]'*)
45+
echo "should_run=true" >> "$GITHUB_OUTPUT"
46+
echo "commit message contains [newver] or [rel] -> run tests"
47+
exit 0
48+
;;
49+
esac
3850
# First push to a branch sends BEFORE as 40 zeros; diff against the
3951
# empty tree in that case so every file counts as added.
4052
if [ -z "$BEFORE" ] || [ "$BEFORE" = "0000000000000000000000000000000000000000" ]; then
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
RANDOM="759382839"
1+
RANDOM="974842911"

0 commit comments

Comments
 (0)