Skip to content

fix: resolve 91 ruff E701/E702 lint violations in commands/ and extractors/#1

Merged
DevanshuNEU merged 1 commit into
mainfrom
fix/ruff-e701-e702
Mar 17, 2026
Merged

fix: resolve 91 ruff E701/E702 lint violations in commands/ and extractors/#1
DevanshuNEU merged 1 commit into
mainfrom
fix/ruff-e701-e702

Conversation

@DevanshuNEU
Copy link
Copy Markdown
Contributor

@DevanshuNEU DevanshuNEU commented Mar 17, 2026

Problem

The refactor that split cli.py and extractor.py into focused modules introduced 91 ruff lint violations across the new files. All compact one-liners written during the split (if x: return y, a.append(x); b.add(x)) fail ruff E701/E702 rules, which require multi-line formatting.

This caused CI to fail on all Python versions and all platforms.

What changed

Expanded all single-line if/else/return statements and semicolon-separated statements to standard multi-line format across 6 files:

  • saar/commands/extract.py
  • saar/commands/quality.py
  • saar/extractors/backend.py
  • saar/extractors/conventions.py
  • saar/extractors/frontend.py
  • saar/extractors/project.py

No logic changes. Pure formatting fix.

Verified locally

ruff check saar/ tests/  →  All checks passed
pytest tests/ -q         →  548 passed, 2 warnings

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
saar Ready Ready Preview, Comment Mar 17, 2026 4:04am

@DevanshuNEU DevanshuNEU changed the title fix: E701/E702 ruff lint errors in commands/ and extractors/ -- expan… fix: resolve 91 ruff E701/E702 lint violations in commands/ and extractors/ Mar 17, 2026
@DevanshuNEU DevanshuNEU merged commit 2bcf68f into main Mar 17, 2026
15 checks passed
@DevanshuNEU DevanshuNEU deleted the fix/ruff-e701-e702 branch March 17, 2026 04:33
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