From 6319c7f577b02f66853feb1c3dd22fb2d6981c5f Mon Sep 17 00:00:00 2001 From: Johnathan Corgan Date: Thu, 23 Jul 2026 15:46:04 +0000 Subject: [PATCH] Describe the trailing command accurately in the gate's finding message For a value-returning helper the trailing echo is the return mechanism, not a log call. The hazard is the same either way -- it fixes the exit status at zero -- but calling it a log call misdescribes four of the sites the gate now finds. --- testing/check-trailing-log.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/testing/check-trailing-log.py b/testing/check-trailing-log.py index b4f4817..990dc34 100755 --- a/testing/check-trailing-log.py +++ b/testing/check-trailing-log.py @@ -195,8 +195,8 @@ def main() -> int: print(f" last statement: {stmt[:100]}") print(f" status consumed by: {', '.join(callers)}") print(f" fix: add an explicit `return 0` as the last statement. Its " - f"success value is currently whatever the log call returned, " - f"which is 0 whatever the function did.") + f"success value is currently the trailing command's, which is 0 " + f"whatever the function did.") print(f"trailing-log check: {scanned} function(s) scanned, " f"{shaped} end in a logging call, {len(ALLOWED)} allowed, "