FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add module ignoring functionality to debugger by ktowen · Pull Request #14973 · ipython/ipython · GitHub

Add module ignoring functionality to debugger - #14973

Merged
krassowski merged 6 commits into
ipython:mainfrom
ktowen:debugger-ignore-modules
Sep 26, 2025
Merged

Add module ignoring functionality to debugger#14973
krassowski merged 6 commits into
ipython:mainfrom
ktowen:debugger-ignore-modules

Conversation

ktowen commented Aug 28, 2025
edited by krassowski
Loading

Copy link
Copy Markdown
Contributor

This PR adds two new commands to ignore (and unignore) specific modules when navigating frames in the IPython debugger, similar to how the debugger already skips hidden frames. It will work with all the commands that implies moving between frames like up, down, step and next.
It is useful when you don't want the debugger to enter third party code.

Closes #1665

ktowen marked this pull request as draft August 28, 2025 11:37
ktowen force-pushed the debugger-ignore-modules branch from f4c9cdb to 5f599d3 Compare August 28, 2025 12:31
ktowen marked this pull request as ready for review August 28, 2025 12:51
krassowski added this to the 9.5 milestone Aug 29, 2025

Copy link
Copy Markdown
Member

I think this closes #1665. The proposal there was to expose skip option from https://docs.python.org/3/library/bdb.html but I think this achieves similar end result.

krassowski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Thank you for the PR @ktowen!

This looks mostly fine, the only thing missing is tests. Can you add tests for both ignore_module and unignore_module with and without wildcard usage?

Comment thread IPython/core/debugger.py Outdated
krassowski modified the milestones: 9.5, 9.6 Aug 29, 2025
ktowen and others added 2 commits August 29, 2025 16:46
Co-authored-by: Michał Krassowski <5832902+krassowski@users.noreply.github.com>
ktowen force-pushed the debugger-ignore-modules branch 3 times, most recently from 1b30a21 to de4bf44 Compare August 30, 2025 12:52
ktowen force-pushed the debugger-ignore-modules branch from de4bf44 to 21de9fc Compare August 30, 2025 14:35
ktowen requested a review from krassowski August 30, 2025 15:04

krassowski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Thank you!

krassowski merged commit cc81ca1 into ipython:main Sep 26, 2025
19 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

continue until reaching your own code

2 participants


Back | FazBrowse Home | New Git URL