-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Added the always_run option in excepthook document page #978
Conversation
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.
Thanks! :)
src/collections/_documentation/platforms/python/default-integrations.md
Outdated
Show resolved
Hide resolved
src/collections/_documentation/platforms/python/default-integrations.md
Outdated
Show resolved
Hide resolved
@untitaker anything else? :) |
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.
Sorry for the back-and-forth!
src/collections/_documentation/platforms/python/default-integrations.md
Outdated
Show resolved
Hide resolved
The learning process for me. I enjoy Review. :) I fixed it. @untitaker |
I will merge this once a new version is released, because this repo deploys on merge. I hope to get to it within this week! Thanks so far. |
@MimiDumpling looping you in based on the meeting we just had. I want to make sure we figure out a process that is not too much friction on external contributors. |
>>> raise Exception("I will become an error") | ||
``` | ||
|
||
By default, the SDK does not capture errors occuring in the REPL (`always_run=False`). |
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.
By default, the SDK does not capture errors occuring in the REPL (`always_run=False`). | |
By default, the SDK does not capture errors occurring in the REPL (`always_run=False`). |
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.
ops, now fixing .. :) Bad typo
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.
No worries! This looks great. :) Thanks for all your work!
Thanks! |
getsentry/sentry-python#364
Is there wrong or missing piece?