-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Fix crash in catching-non-exception when something besides a class in except
#10232
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
Fix crash in catching-non-exception when something besides a class in except
#10232
Conversation
Codecov Reportβ
All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #10232 +/- ##
=======================================
Coverage 95.84% 95.84%
=======================================
Files 175 175
Lines 19058 19058
=======================================
Hits 18266 18266
Misses 792 792
π New features to boost your workflow:
|
This comment has been minimized.
This comment has been minimized.
catching-non-exception when something besides a class in except
jacobtylerwalls
left a comment
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 for the fix!
This comment has been minimized.
This comment has been minimized.
jacobtylerwalls
left a comment
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.
Fixing last push
35f1846 to
4caec3a
Compare
|
π€ According to the primer, this change has no effect on the checked open source code. π€π This comment was generated for commit 4caec3a |
β¦ except (#10232) (#10234) (cherry picked from commit 007a745) Co-authored-by: ChandanChainani <[email protected]>
Type of Changes
Description
Closes #10106