-
Notifications
You must be signed in to change notification settings - Fork 115
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
Make raise facades macros #922
Conversation
✅ Deploy Preview for elastic-ritchie-8f47f9 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Obligatory demo:
|
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.
where: (exception-context message) is so simple yet such a great feature!!
(let (message | ||
(if (string? message) | ||
message | ||
(call-with-output-string "" (cut display message <>)))) |
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.
this is a sanity check, as we send the message over the wire for actor error results and a bug could end up something unintended; this is guaranteed to be a string now.
Adds macros
raise/context
anddefraise/context
to aid capturing exception contexts, and uses them in the stdlib.So that we have exception context everywhere!