-
Notifications
You must be signed in to change notification settings - Fork 1.4k
add post-mortem pdb flag #71
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
base: master
Are you sure you want to change the base?
Conversation
I definitely see the value of this feature. Here are the reasons for my hesitation:
All of that said, this does seem really useful. Perhaps after I try it out for a few days I'll be persuaded its usefulness outweighs the small cost. I'm also interested to see how easy it is to do post-mortem debugging w/o modifying fire, e.g. by try/excepting the call to Fire and using pdb in the except block there. |
never heard about Fire before (used argparse and click extensively), was told about it, saw this issue which loved since nearly the first thing I always need to add such --pdb or --dbg flag. And indeed it is possible to make it all work "manually", and even without try/except, just by overloading But even though this easy, I think making it even easier to enable such "debug" option, would be quite cool. |
this adds an additional fire flag (-- -d|--debug) which provides for automatically dropping a user into post mortem pdb session on exception in the underlying exposed callable.