-
Notifications
You must be signed in to change notification settings - Fork 229
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
Post-mortem debugging in excepthook
no longer works.
#552
Comments
Is this still happening? Trying to figure out if post-mortem debugging is expected to work or not. |
It is if you submit a PR? 🙂 |
@inducer Makes sense, thank you for confirming. I'm happy to try, but I don't know enough about why f77332c was made in the first place. Do you have any pointers on what the correct behavior should be, or what should be tested to make sure I don't break something else? Or should I just revert that commit? |
TBH, I don't remember either. If you can make it work based on your testing, I think that's probably a step in the right direction. It's not a feature I use, so I'm not surprised it bitrotted. |
I have been missing the post-mortem debugging for a while, really hope to see it back! |
See discussion in #549. PRs welcome.
cc @presto-dev
The text was updated successfully, but these errors were encountered: