jazzdotdev / jazz

The Scripting Engine that Combines Speed, Safety, and Simplicity
Apache License 2.0
146 stars 11 forks source link

replace human panic email contact method with Discord link #216

Open naturallymitchell opened 5 years ago

naturallymitchell commented 5 years ago

since #133 went well and I don't like email, let's make this better

dariusc93 commented 5 years ago

So you want the discord link in place of the email or do you want the link to be listed in the panic along with the email?

naturallymitchell commented 5 years ago

Contact methods should include:

naturallymitchell commented 5 years ago

further described in https://github.com/rust-clique/human-panic/issues/7#issuecomment-383035397

naturallymitchell commented 5 years ago

https://github.com/rust-clique/human-panic/blob/master/src/lib.rs#L162-L173

naturallymitchell commented 5 years ago

I think instead of forking it, which sucks because then we have to push that to crates.io, or fixing it upstream, we can use this https://doc.rust-lang.org/cargo/reference/manifest.html#the-patch-section

dariusc93 commented 5 years ago

Relates to 83423689d2b9e39dbd0cdc841399e9d769f23f65