Skip to content

refactor: move wingman to src/ with linting fixes#9

Open
Chesars wants to merge 2 commits intodedalus-labs:mainfrom
Chesars:refactor/directory-structure-with-linting
Open

refactor: move wingman to src/ with linting fixes#9
Chesars wants to merge 2 commits intodedalus-labs:mainfrom
Chesars:refactor/directory-structure-with-linting

Conversation

@Chesars
Copy link

@Chesars Chesars commented Dec 27, 2025

Description

Fixed linting errors in src/wingman/ directory from PR #6.

Details

  • Organized and cleaned up imports (removed unused imports)
  • Removed unnecessary f-string prefixes
  • Simplified code structure (combined nested ifs, replaced try-except-pass with contextlib.suppress)
  • Used more pythonic patterns (enumerate instead of manual counter, ternary operators)
  • Fixed ambiguous variable name (lline)

All changes are style-only and preserve existing behavior. No logic changes.

✅ uv run ruff check src/wingman/     # All checks passed!
✅ uv run ruff format --check src/wingman/  # 15 files already formatted

Type of Change

  • Refactoring (no functional changes)

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • My changes generate no new warnings
  • I have tested the TUI locally

@aryanma aryanma force-pushed the main branch 2 times, most recently from f33d593 to 733080e Compare January 8, 2026 03:52
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