◐ Shell
clean mode source ↗

Add bpdb to debug_statements_hook by mwip · Pull Request #942 · pre-commit/pre-commit-hooks

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@mwip

@mwip mwip commented

Aug 2, 2023

Copy link Copy Markdown

Contributor

This patch adds the bpython debugger import bpdb to the debug_statements_hook.

Closes #941

This patch adds the bpython debugger import `bpdb` to the
debug_statements_hook.

@asottile asottile 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.

@asottile asottile merged commit 1790c6b into pre-commit:main

Aug 2, 2023

logpie added a commit to logpie/otto that referenced this pull request

Mar 23, 2026
All 10 projects validated: verify.sh FAILs on BEFORE commit, passes on AFTER.

Python projects (8):
- real-dotenv-cwd-bug: Fix CLI crash when CWD deleted (theskumar/python-dotenv#446)
- real-humanize-precisedelta: Fix rounding carry-over in precisedelta (python-humanize/humanize#254)
- real-humanize-intword: Fix plural form and boundary rounding in intword (python-humanize/humanize#273)
- real-iniconfig-parse: Add IniConfig.parse() with inline comment stripping (pytest-dev/iniconfig#70)
- real-itsdangerous-overflow: Handle date overflow in timed unsign (pallets/itsdangerous)
- real-pathspec-empty: Fix ValueError for empty pattern lists in backends (cpburnz/python-pathspec#100)
- real-precommit-empty-sort: Fix file-contents-sorter adding blank line to empty files (pre-commit/pre-commit-hooks#935)
- real-precommit-debug-bpdb: Add bpdb debugger detection (pre-commit/pre-commit-hooks#942)

Node.js projects (2):
- real-yargs-parser-hyphens: Fix parsing of options ending with 3+ hyphens (yargs/yargs-parser#434)
- real-string-width-emoji: Fix width for minimally-qualified emoji sequences (sindresorhus/string-width#68)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@asottile asottile asottile approved these changes

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

bpython debugger not caught by debug-statements-hook

2 participants

@mwip @asottile