Merge pull request #432 from Yhg1s/fix-logging

Fix logging groups for GitHub output so they collapse correctly.

Author: mdboom

bench_runner/templates/workflow_bootstrap.src.py

@@ -59,12 +59,12 @@ def install_requirements(venv: Path) -> None:
 
 def main():
     venv = Path("venv")
-    print("::group::Creating venv")
+    print("::group::Creating venv", file=sys.stderr)
     create_venv(venv)
-    print("::endgroup::")
-    print("::group::Installing requirements")
+    print("::endgroup::", file=sys.stderr)
+    print("::group::Installing requirements", file=sys.stderr)
     install_requirements(venv)
-    print("::endgroup::")
+    print("::endgroup::", file=sys.stderr)
 
     # Now that we've installed the full bench_runner library,
     # continue on in a new process...

bench_runner/util.py

@@ -111,11 +111,11 @@ def format_seconds(value: float) -> str:
 
     @contextlib.contextmanager
     def log_group(text: str) -> Iterator:
-        print(f"::group::{text}")
+        print(f"::group::{text}", file=sys.stderr)
         try:
             yield
         finally:
-            print("::endgroup::")
+            print("::endgroup::", file=sys.stderr)
 
     def track(iterable: Iterable, name: str) -> Iterable:
         with log_group(name):