Skip to content

Enforce ruff/pyupgrade rule UP031#12137

Merged
jacobtomlinson merged 7 commits intodask:mainfrom
DimitriPapadopoulos:UP031
Nov 11, 2025
Merged

Enforce ruff/pyupgrade rule UP031#12137
jacobtomlinson merged 7 commits intodask:mainfrom
DimitriPapadopoulos:UP031

Conversation

@DimitriPapadopoulos
Copy link
Copy Markdown
Contributor

UP031 Use format specifiers instead of percent format

The first commit contains (unsafe) automatic fixes:

$ ruff check --fix --unsafe-fixes
UP031 Use format specifiers instead of percent format
...
Found 266 errors (224 fixed, 42 remaining).
$
  • Passes pre-commit run --all-files

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 7, 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 9m 21s ⏱️ - 6m 25s
 18 154 tests ±0   16 939 ✅ ±0   1 215 💤 ±0  0 ❌ ±0 
162 614 runs  ±0  150 520 ✅  - 2  12 094 💤 +2  0 ❌ ±0 

Results for commit 5d24891. ± Comparison against base commit 6708b43.

♻️ This comment has been updated with latest results.

@DimitriPapadopoulos DimitriPapadopoulos force-pushed the UP031 branch 8 times, most recently from a9c81dc to cacc555 Compare November 8, 2025 16:08
UP031 Use format specifiers instead of percent format

This commit contains (unsafe) automatic fixes:
	$ ruff check --fix --unsafe-fixes
	UP031 Use format specifiers instead of percent format
	...
	Found 266 errors (224 fixed, 42 remaining).
	$
Do that multiple times. Eventually they converge to this stable result.
RUF010 Use explicit conversion flag
Converting to string is the default.
UP031 Use format specifiers instead of percent format

This commit contains only manual fixes.
UP031 Use format specifiers instead of percent format
@jacobtomlinson jacobtomlinson merged commit d5e8ed4 into dask:main Nov 11, 2025
24 of 25 checks passed
@DimitriPapadopoulos DimitriPapadopoulos deleted the UP031 branch November 11, 2025 18:00
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