-
Notifications
You must be signed in to change notification settings - Fork 7
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
:ret
spec not being instrumented
#16
Comments
Hi @thiru. You're correct - that's the expected behavior now. I opted to simplify this library's behavior (and thus maintainability) in #9 by removing the validity checking code. It now uses the built-in |
I see, yeah that's totally understandable. But maybe it's worth adding a note in the readme? As it is I suppose others may get confused as I did. |
Happy to submit a PR with a proposed changed if you'd be open to it? |
That'd be great, thanks! |
Just added a simple note to the readme: #17 |
I followed along with your example in the readme and everything is working as expected except that the
:ret
spec formy-inc
isn't being instrumented.s/*compile-asserts*
istrue
and the:args
spec is instrumenting correctly.The text was updated successfully, but these errors were encountered: