Skip to content
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

Closed
wants to merge 1 commit into from

Conversation

jbms
Copy link
Contributor

@jbms jbms commented Aug 2, 2024

Feature or Bugfix

  • Feature

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.

@jbms jbms requested a review from AA-Turner August 2, 2024 17:42
@jayaddison jayaddison added type:enhancement enhance or introduce a new feature internals:other labels Aug 4, 2024
Copy link
Member

@picnixz picnixz left a 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?

CHANGES.rst Outdated Show resolved Hide resolved
@jbms
Copy link
Contributor Author

jbms commented Aug 6, 2024

Could we have some tests? or is it too hard to test and mock pdb?

Added a test.

Copy link
Member

@picnixz picnixz left a 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)

tests/test_builders/test_build_warnings.py Outdated Show resolved Hide resolved
CHANGES.rst Outdated Show resolved Hide resolved
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.
@AA-Turner
Copy link
Member

Closing, as I believe that this is covered by #12743. Happy to reopen if I'm wrong, though.

A

@AA-Turner AA-Turner closed this Aug 14, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
internals:other type:enhancement enhance or introduce a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants