Skip to content

Replace pre-commit hook with ruff rule#12142

Merged
jacobtomlinson merged 1 commit intodask:mainfrom
DimitriPapadopoulos:pre-commit-hooks
Nov 10, 2025
Merged

Replace pre-commit hook with ruff rule#12142
jacobtomlinson merged 1 commit intodask:mainfrom
DimitriPapadopoulos:pre-commit-hooks

Conversation

@DimitriPapadopoulos
Copy link
Copy Markdown
Contributor

flake8-debugger rule is equivalent to debug-statements pre-commit hook.

@DimitriPapadopoulos DimitriPapadopoulos changed the title replace pre-commit hook with ruff rule Replace pre-commit hook with ruff rule Nov 8, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 8, 2025

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

      9 files  ±0        9 suites  ±0   3h 16m 40s ⏱️ + 4m 47s
 18 154 tests ±0   16 939 ✅ ±0   1 215 💤 ±0  0 ❌ ±0 
162 614 runs  ±0  150 521 ✅ ±0  12 093 💤 ±0  0 ❌ ±0 

Results for commit d69b6ed. ± Comparison against base commit a66a897.

♻️ This comment has been updated with latest results.

@jacobtomlinson
Copy link
Copy Markdown
Member

@DimitriPapadopoulos can you resolve the conflicts here

flake8-debugger rule is equivalent to debug-statements pre-commit hook.
@jacobtomlinson jacobtomlinson merged commit 6708b43 into dask:main Nov 10, 2025
24 of 25 checks passed
@DimitriPapadopoulos DimitriPapadopoulos deleted the pre-commit-hooks branch November 10, 2025 12:18
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.

2 participants