Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Error reporting QoL #88

Merged
merged 2 commits into from
Aug 13, 2024
Merged

Error reporting QoL #88

merged 2 commits into from
Aug 13, 2024

Commits on Aug 13, 2024

  1. Error reporting QoL 

    - require Display instead of Debug for Actor::Error associated type
- print handle() (and friends) errors using `{:#}` rather than `{:?}`
  - this hides the backtrace print of anyhow errors
- differentiate handle() (and friends) errors while shutting down vs. running and log them with lower severity
- make it more clear and less spammy when trying to shut down a system while it is already shutting down
    @strohel
    strohel committed Aug 13, 2024
    Configuration menu
    Copy the full SHA
    30941a3 View commit details
    Browse the repository at this point in the history

  2. Tweak error, log message punctuation, casing, prefixes 

    Coming from live testing the these changes.
    @strohel
    strohel committed Aug 13, 2024
    Configuration menu
    Copy the full SHA
    0df2774 View commit details
    Browse the repository at this point in the history