Skip to content

MAINT: ruff check#1420

Merged
oesteban merged 15 commits intonipreps:masterfrom
DimitriPapadopoulos:ruff
Jan 26, 2026
Merged

MAINT: ruff check#1420
oesteban merged 15 commits intonipreps:masterfrom
DimitriPapadopoulos:ruff

Conversation

@DimitriPapadopoulos
Copy link
Copy Markdown
Contributor

@DimitriPapadopoulos DimitriPapadopoulos commented Sep 17, 2025

ruff check had been left out from tests.

- No need to specify `F` and `E` rules with `extend-select`.
- Update rules to ignore.
E741 Ambiguous variable name: `l`
E401 Multiple imports on one line
W605 Invalid escape sequence
I001 Import block is un-sorted or un-formatted
C408 Unnecessary `dict()` call (rewrite as a literal)
C413 Unnecessary `list()` call around `sorted()`
C414 Unnecessary `list()` call within `sorted()`
C420 Unnecessary dict comprehension for iterable; use `dict.fromkeys` instead
PT001 Use `@pytest.fixture` over `@pytest.fixture()`
UP031 Use format specifiers instead of percent format
UP035 Import from `collections.abc` instead
ICN001 `seaborn` should be imported as `sns`
@oesteban oesteban merged commit 7a69ea2 into nipreps:master Jan 26, 2026
9 of 11 checks passed
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