Skip to content

Commit 6de6d6a

Browse files
authored
chore: replace debug-statements hook with ruff T100 (#2954)
# Rationale for this change Like #2953, doing a dive into the linted dependencies we can replace the `debug-statements` pre-commit hook with ruff's [`T100` rule flake8-debugger](https://docs.astral.sh/ruff/rules/debugger/). This will lighten the linters needed when working on cb. T100 catches more modern debuggers (`debugpy`, `ptvsd`, `IPython.embed`). It seems like there are rare debuggers it misses but aren't used like `bpdb`. ## Are these changes tested? `make lint` ``` > ruff check --select T100 pyiceberg/ tests/ All checks passed! ``` ## Are there any user-facing changes? No
1 parent 4769d07 commit 6de6d6a

2 files changed

Lines changed: 1 addition & 1 deletion

File tree

.pre-commit-config.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ repos:
2323
hooks:
2424
- id: trailing-whitespace
2525
- id: end-of-file-fixer
26-
- id: debug-statements
2726
- id: check-yaml
2827
- id: check-ast
2928
- repo: https://github.com/astral-sh/ruff-pre-commit

ruff.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,7 @@ select = [
5656
"C4", # flake8-comprehensions
5757
"I", # isort
5858
"UP", # pyupgrade
59+
"T100", # flake8-debugger
5960
]
6061
ignore = []
6162

0 commit comments

Comments
 (0)