Skip to content
This repository has been archived by the owner on Sep 22, 2020. It is now read-only.

#[derive(MessageResponse)] #13

Closed
rotty opened this issue Jan 8, 2019 · 1 comment
Closed

#[derive(MessageResponse)] #13

rotty opened this issue Jan 8, 2019 · 1 comment

Comments

@rotty
Copy link

rotty commented Jan 8, 2019

I think it would make sense to add a MessageResponse custom derive, along the lines of the SIMPLE_RESULT macro. I know there's the MessageResult helper, but I prefer not having to wrap my result types.

I've read through the issues #1 and #2, and understand the reluctance to add new macros if they only save a small amount of code, but in this case (as shown by the existence of SIMPLE_RESULT ;-)), I think the boilerplate saved is enough to justify a custom derive.

@JohnTitor
Copy link
Member

I think we can close this issue by #14.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants