-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Allow --pdb option to also debug warnings #12727
Conversation
11e1a85
to
de19354
Compare
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.
Could we have some tests? or is it too hard to test and mock pdb?
de19354
to
9954c49
Compare
Added a test. |
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.
Minor nitpicks while taking my breakfast (and thanks for the commit)
9954c49
to
ed86a6e
Compare
Previously, warnings emitted during reading and writing were deferred, which made --pdb ineffective for debugging them. With this change, warnings are no longer deferred when --pdb and --fail-on-warning are both specified.
ed86a6e
to
b318a66
Compare
Closing, as I believe that this is covered by #12743. Happy to reopen if I'm wrong, though. A |
Feature or Bugfix
Purpose
Previously, warnings emitted during reading and writing were deferred, which made --pdb ineffective for debugging them.
With this change, warnings are no longer deferred when --pdb and --fail-on-warning are both specified.