-
Notifications
You must be signed in to change notification settings - Fork 508
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
feat: Relax Message Debug trait bound #1147
base: master
Are you sure you want to change the base?
Conversation
9fb1f4f
to
6bac6c3
Compare
6bac6c3
to
28e633b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Can you explain why this is useful?
- Is this an API breaking change?
When the
I think this is a breaking change. Previously, when a value's type had a |
I see, so
I understand. I will prepare this for the next breaking release |
Relaxes
Message
'sDebug
trait bound.BREAKING CHANGE:
trait Debug
was a supertrait oftrait Message
. This is no longer required byprost
. If your code relies ontrait Debug
being implemented for everyimpl Message
, you must now explicitly state that you require both Debug and Message. For example:where M: Debug + Message