Skip to content

style(lint): fix ruff E501, F401, F841, SIM102, SIM117, I001#29

Merged
subkoks merged 1 commit into
mainfrom
style/lint-fixes
Jul 16, 2026
Merged

style(lint): fix ruff E501, F401, F841, SIM102, SIM117, I001#29
subkoks merged 1 commit into
mainfrom
style/lint-fixes

Conversation

@subkoks

@subkoks subkoks commented Jul 16, 2026

Copy link
Copy Markdown
Owner

Fixes 25 ruff lint violations across src/ and tests/:

  • E501 — Break long lines in regex patterns and comments
  • F401/F841 — Remove unused tqdm import, unused organized_dir variable
  • SIM102 — Combine nested if to single conditional
  • SIM117 — Combine nested with statements using single context
  • I001 — Fix import ordering in all test files
  • F541 — Remove extraneous f-string prefix

All 120 tests pass. Ruff lint: clean.

…rc and tests

- Fix E501 line-too-long violations in organize_downloads.py and tg_schematic_downloader.py
- Remove unused tqdm import and unused organized_dir variable
- Combine nested if/with statements per SIM102/SIM117
- Fix import ordering I001 in all test files
- Remove unused imports (pytest, os) in test files
@subkoks
subkoks enabled auto-merge (squash) July 16, 2026 17:58
@subkoks
subkoks merged commit a6bcd0d into main Jul 16, 2026
3 of 4 checks passed
@subkoks
subkoks deleted the style/lint-fixes branch July 16, 2026 17:58
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.

1 participant