Skip to content

Commit

Permalink
Improve dmypy status message to clarify state
Browse files Browse the repository at this point in the history
Update the status message to clarify that the daemon may be busy and not necessarily stuck. The new message provides better guidance to users without prematurely suggesting killing the daemon.

Fixes python#18008
  • Loading branch information
kcornw committed Dec 23, 2024
1 parent ceaf48d commit 376b99d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions mypy/dmypy/client.py
Original file line number Diff line number Diff line change
Expand Up @@ -436,7 +436,7 @@ def do_status(args: argparse.Namespace) -> None:
if args.verbose or "error" in response:
show_stats(response)
if "error" in response:
fail(f"Daemon is stuck; consider {sys.argv[0]} kill")
fail(f"Daemon may be busy processing; if this persists, consider {sys.argv[0]} kill")
print("Daemon is up and running")


Expand All @@ -447,7 +447,7 @@ def do_stop(args: argparse.Namespace) -> None:
response = request(args.status_file, "stop", timeout=5)
if "error" in response:
show_stats(response)
fail(f"Daemon is stuck; consider {sys.argv[0]} kill")
fail(f"Daemon may be busy processing; if this persists, consider {sys.argv[0]} kill")
else:
print("Daemon stopped")

Expand Down

0 comments on commit 376b99d

Please sign in to comment.