Replace derive_more

[svix-jplatte]

Hey, I see that derive_more is only used in two modules to derive Display. With many proc-macros having moved to syn 2.0 but derive_more not (outside of beta releases), plus being somewhat heavy itself (containing much more than just that Display derive, taking 3.3s to compile on my machine compared to syn's 6s), I think this dependency is a rather big compile time hit for the relatively small value it provides.

Would you accept a PR that replaced all four derived Display implementations with hand-written ones? Alternatively, would you accept a PR that replaced the three Display derives (and two missing Error impls) in error.rs with thiserror::Error derives, and the last Display derive with a manual implementation?

[bartlomieju]

Hey @svix-jplatte, thanks for spotting this. We will definitely accept a PR that removes derive_more in favor of using a hand rolled solution.

[svix-jplatte]

PR is up ^