Conversation
vriesk
left a comment
There was a problem hiding this comment.
Hi,
This should work, though the need to invoke additional macro is somewhat clunky.
Also, in case of pretty_assertions, it does not have its own assert! macro, only assert_eq! and assert_ne!. I'm unsure what's the use case for overriding of assert! is.
|
Yeah, it's clunky, however there's no better way without providing custom test harness. As for |
|
|
|
I'm gonna fix tests in this and push it then. I'd really avoid feature flags, as they tend to break when |
Closes #109
Closes #84