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.

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

Use human-panic? #887

Closed
spenserblack opened this issue Dec 2, 2022 · 3 comments
Closed

Use human-panic? #887

spenserblack opened this issue Dec 2, 2022 · 3 comments
Labels
enhancement New feature or request

Comments

@spenserblack
Copy link
Collaborator

What do you think about using human-panic for prettier panics?

In a --release build, this hook will change the message to

Well, this is embarrassing.

{{ crate_name }} had a problem and crashed. To help us diagnose the problem you can send us a crash report.

e have generated a report file at "/tmp/{{ filename }}.toml". Submit an issue or email with the subject of "{{ crate_name }} Crash Report" and include the report as an attachment.

We take privacy seriously, and do not perform any automated error collection. In order to improve the software, we rely on people to submit reports.

Thank you kindly!

The hook also generates the named crash report on a panic.

@o2sh o2sh added the enhancement New feature or request label Dec 3, 2022
@o2sh
Copy link
Owner

o2sh commented Dec 3, 2022

I love it, quite useful and easy to setup. 👍

o2sh added a commit that referenced this issue Dec 3, 2022
@o2sh
Copy link
Owner

o2sh commented Dec 3, 2022

with 975856d

image

@spenserblack
Copy link
Collaborator Author

Looks great!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants