From df6d98799c4d096899206a35cc4ee7aa93d63b93 Mon Sep 17 00:00:00 2001 From: Gergely Nagy Date: Sat, 9 Dec 2023 13:25:27 +0100 Subject: [PATCH] [GITEA] 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 (cherry picked from commit d75f13bde7c2242b99c51f9d51e1209b13d34c1f) --- 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 }