Merge pull request #9 from nsheaps/fix/running-ci-emoji

fix: show running CI as 🟡 when non-required checks still in progress

Author: nsheaps

.github/workflows/test.yaml

@@ -0,0 +1,33 @@
+name: Test
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Check bash syntax
+        run: |
+          for script in bin/*; do
+            [[ -d "$script" ]] && continue
+            echo "Checking $script..."
+            bash -n "$script"
+          done
+          echo "All scripts passed syntax check."
+
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Setup mise
+        uses: jdx/mise-action@v2
+
+      - name: Run tests
+        run: mise run test

bin/gs-stack-status

@@ -430,25 +430,21 @@ if [[ -n "$repo_owner" && ${#pr_number_to_branch[@]} -gt 0 ]]; then
           else $valid
           end) as $filtered |
 
-          # CheckRun nodes: running if status exists and != COMPLETED
-          ($filtered | map(select(
-            .status != null and .status != "COMPLETED"
-          )) | length) as $cr_running |
-          # CheckRun nodes: failed conclusions
+          # any_running: use ALL checks (not just filtered/required) so
+          # non-required in-progress checks keep overall status as pending
+          (([$valid[] | select(
+            (.status != null and .status != "COMPLETED") or
+            (.cState != null and (.cState | IN("PENDING", "EXPECTED")))
+          )] | length) > 0) as $any_running |
+          # Failed: only from filtered (required) checks
           ($filtered | map(select(
             .conclusion != null and
             (.conclusion | IN("FAILURE", "TIMED_OUT", "ERROR", "STARTUP_FAILURE", "ACTION_REQUIRED"))
           )) | length) as $cr_failed |
-          # StatusContext nodes: running (PENDING/EXPECTED)
-          ($filtered | map(select(
-            .cState != null and (.cState | IN("PENDING", "EXPECTED"))
-          )) | length) as $sc_running |
-          # StatusContext nodes: failed
           ($filtered | map(select(
             .cState != null and (.cState | IN("FAILURE", "ERROR"))
           )) | length) as $sc_failed |
 
-          (($cr_running + $sc_running) > 0) as $any_running |
           (($cr_failed + $sc_failed) > 0) as $any_failed |
 
           if $any_running and $any_failed then