Skip to content

Fix ruff lint errors and migrate config format - #74

Open
ogajduse wants to merge 1 commit into
SatelliteQE:masterfrom
ogajduse:fix-ruff-lint-errors
Open

ogajduse wants to merge 1 commit into
SatelliteQE:masterfrom
ogajduse:fix-ruff-lint-errors

Conversation

@ogajduse

@ogajduse ogajduse commented Jun 17, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Move import requests to module level in manifester.py (fixes PLC0415)
  • Migrate ruff config from deprecated top-level format to lint.* sections
  • Update remapped rule codes (TRY200→B904, TRY302→TRY203)
  • Remove PGH001 from ignore list — it was remapped to S307 which is already in select, so the eval check is now active (previously contradictory: selected and ignored simultaneously)
  • Increase max-branches to 20 to match max-complexity
  • Apply ruff format to all files

Test plan

  • ruff check . passes
  • ruff format --check . passes
  • All 13 tests pass

🤖 Generated with Claude Code

Move `import requests` to module level in manifester.py to fix
PLC0415 violations. Migrate ruff config from deprecated top-level
format to lint.* sections. Update remapped rule codes (TRY200->B904,
TRY302->TRY203). Increase max-branches to 20 to match max-complexity.
Apply ruff format to all files.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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