Skip to content

Commit b9c3bc1

Browse files
SentienceDEVclaude
andcommitted
Fix: grep exit code in verify source code step
The previous || pattern didn't properly handle grep's exit code. Using if/else ensures correct exit behavior: - Exit 0 when assertTrue is NOT found (correct) - Exit 1 when assertTrue IS found (error) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 59fd770 commit b9c3bc1

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

.github/workflows/test.yml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,12 @@ jobs:
4141
echo "=== Verifying assert_ method exists ==="
4242
grep -n "def assert_" sentience/agent_runtime.py
4343
echo "=== Checking for assertTrue (should NOT exist) ==="
44-
grep -n "assertTrue" sentience/agent_runtime.py || echo "Good: no assertTrue found"
44+
if grep -n "assertTrue" sentience/agent_runtime.py; then
45+
echo "ERROR: Found assertTrue - this should have been removed!"
46+
exit 1
47+
else
48+
echo "Good: no assertTrue found"
49+
fi
4550
4651
- name: Lint with pre-commit
4752
continue-on-error: true

0 commit comments

Comments
 (0)