Incorporate feedback

Author: georgesittas

sqlmesh/core/console.py

@@ -1150,10 +1150,13 @@ def update_signal_progress(
 
         # Color coding to help detect partial interval ranges quickly
         if ready_intervals == check_intervals:
+            msg = "All ready"
             color = "green"
         elif ready_intervals:
+            msg = "Some ready"
             color = "yellow"
         else:
+            msg = "None ready"
             color = "red"
 
         if self.verbosity < Verbosity.VERY_VERBOSE:
@@ -1168,20 +1171,20 @@ def update_signal_progress(
                 formatted_ready_intervals.append(f"... and {num_ready_intervals - 3} more")
 
             check = ", ".join(formatted_check_intervals)
-            tree.add(f"check: {check}")
+            tree.add(f"Check: {check}")
 
             ready = ", ".join(formatted_ready_intervals)
-            tree.add(f"[{color}]ready: {ready}[/{color}]")
+            tree.add(f"[{color}]{msg}: {ready}[/{color}]")
         else:
-            check_tree = Tree("check")
+            check_tree = Tree("Check")
             tree.add(check_tree)
             for interval in formatted_check_intervals:
                 check_tree.add(interval)
 
-            ready_tree = Tree(f"[{color}]ready[/{color}]")
+            ready_tree = Tree(f"[{color}]{msg}[/{color}]")
             tree.add(ready_tree)
             for interval in formatted_ready_intervals:
-                ready_tree.add(interval)
+                ready_tree.add(f"[{color}]{interval}[/{color}]")
 
         if self.signal_status_tree is not None:
             self.signal_status_tree.add(tree)

tests/cli/test_cli.py

@@ -2141,12 +2141,12 @@ def none_ready(batch):
 
     assert "Checking signals for sqlmesh_example.model_with_signals" in result.output
     assert "[1/1] only_first_two_ready" in result.output
-    assert "check: 2022-12-28 - 2022-12-31" in result.output
-    assert "ready: 2022-12-28 - 2022-12-29" in result.output
+    assert "Check: 2022-12-28 - 2022-12-31" in result.output
+    assert "Some ready: 2022-12-28 - 2022-12-29" in result.output
 
     assert "Checking signals for sqlmesh_example.model_with_unready" in result.output
     assert "[1/1] none_ready" in result.output
-    assert "ready: no intervals" in result.output
+    assert "None ready: no intervals" in result.output
 
     # Test 2: Run command with start and end dates
     result = runner.invoke(
@@ -2165,13 +2165,13 @@ def none_ready(batch):
 
     assert "Checking signals for sqlmesh_example.model_with_signals" in result.output
     assert "[1/1] only_first_two_ready" in result.output
-    assert "check: 2022-12-30 - 2022-12-31" in result.output
-    assert "ready: 2022-12-30 - 2022-12-31" in result.output
+    assert "Check: 2022-12-30 - 2022-12-31" in result.output
+    assert "All ready: 2022-12-30 - 2022-12-31" in result.output
 
     assert "Checking signals for sqlmesh_example.model_with_unready" in result.output
     assert "[1/1] none_ready" in result.output
-    assert "check: 2022-12-29 - 2022-12-31" in result.output
-    assert "ready: no intervals" in result.output
+    assert "Check: 2022-12-29 - 2022-12-31" in result.output
+    assert "None ready: no intervals" in result.output
 
     # Only one model was executed
     assert "100.0% • 1/1 • 0:00:00" in result.output