9ca46967d9
Merge pull request 'hotfix(cli): replace raw subprocess tracebacks with styled error messages and fix restart race condition' (#96) from hotfix-restart-race-condition into main
Changed it.
Resolved in my book. Inspected the difference as well.
Is that not okay for a rare command as restart?
We could change the loop to check every 250ms instead, so it is 10 checks instead of 50.
You may resolve conversations if satisfied.
It was to have separate tests. Simpler in my eyes (at least at time of writing). I can simplify it by having a start_instance() that start uses outright, and is reused by restart. This could…
Is that not an messier implementation? A user would want a restart to be quick (quicker than running the kill and start command separately). 2.5 or higher stock sleep timer would make little…
Change occurred since I wanted an error block such as in the screenshot. But with that info, ipc is indeed not needed in the code.
Was not aware of this. Remaining functions use ipc, which is why I did not bat an eye. Resolving now.
Since the text formatting does not show it well, here is an image.