From d75f13bde7c2242b99c51f9d51e1209b13d34c1f Mon Sep 17 00:00:00 2001 From: Gergely Nagy Date: Sat, 9 Dec 2023 13:25:27 +0100 Subject: [PATCH] doctor: Don't say All done when no checks were run When there were no checks run, don't say "All done", but "No checks to run" instead, to make it clear what happened. Addresses another part of #1905. Signed-off-by: Gergely Nagy --- modules/doctor/doctor.go | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/modules/doctor/doctor.go b/modules/doctor/doctor.go index ceee322852..4f092ff4b5 100644 --- a/modules/doctor/doctor.go +++ b/modules/doctor/doctor.go @@ -104,7 +104,11 @@ func RunChecks(ctx context.Context, colorize, autofix bool, checks []*Check) err logger.Info("OK") } } - logger.Info("\nAll done.") + if len(checks) > 0 { + logger.Info("\nAll done.") + } else { + logger.Info("\nNo checks to run.") + } return nil }