Skip to content

Commit aede7a8

Browse files
committed
fix: Correct awk comparison syntax in coverage summary step
1 parent 3faa738 commit aede7a8

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

.github/workflows/bindings-sysman-python-unit-tests-coverage.yml

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -251,8 +251,13 @@ jobs:
251251
echo "| Current Coverage | ${{ steps.coverage.outputs.coverage_pct }}% |" >> $GITHUB_STEP_SUMMARY
252252
echo "| Target Branch Coverage | ${{ steps.baseline.outputs.baseline_coverage }}% |" >> $GITHUB_STEP_SUMMARY
253253
254-
THRESHOLD="${{ steps.baseline.outputs.baseline_coverage }}"
255-
if [ $(echo "${{ steps.coverage.outputs.coverage_pct }} >= $THRESHOLD" | awk '{print ($1 >= $3)}') -eq 1 ]; then
254+
CURRENT_COV="${{ steps.coverage.outputs.coverage_pct }}"
255+
BASELINE_COV="${{ steps.baseline.outputs.baseline_coverage }}"
256+
257+
# Use awk for comparison (handles empty/zero baseline gracefully)
258+
if [ -z "$BASELINE_COV" ] || [ "$BASELINE_COV" == "0" ]; then
259+
echo "| Status | ✅ PASSED (Baseline Established) |" >> $GITHUB_STEP_SUMMARY
260+
elif awk "BEGIN {exit !($CURRENT_COV >= $BASELINE_COV)}"; then
256261
echo "| Status | ✅ PASSED |" >> $GITHUB_STEP_SUMMARY
257262
else
258263
echo "| Status | ❌ FAILED |" >> $GITHUB_STEP_SUMMARY

0 commit comments

Comments
 (0)