Skip to content

fix(ci): resolve ruff linter and format errors#15

Merged
abrichr merged 1 commit into
mainfrom
fix/ci-ruff-errors
Jan 27, 2026
Merged

fix(ci): resolve ruff linter and format errors#15
abrichr merged 1 commit into
mainfrom
fix/ci-ruff-errors

Conversation

@abrichr
Copy link
Copy Markdown
Member

@abrichr abrichr commented Jan 27, 2026

Summary

Fix CI failures by resolving all ruff linter and format errors.

Changes

  • E402 (import order): Move warnings.warn() after imports in deprecated viewer files
  • F401 (unused imports): Remove unused Any, base64, os, Service imports
  • F541 (f-string): Remove f-string prefix where no placeholders exist
  • Formatting: Apply ruff format to 5 files with style inconsistencies

Files Changed (7)

File Issue Fixed
benchmarks/viewer.py E402
training/viewer.py E402
benchmarks/waa_deploy/api_agent.py F401 + format
scripts/capture_screenshots.py F401, F541 + format
benchmarks/azure_ops_tracker.py format only
benchmarks/vm_monitor.py format only
cloud/local.py format only

Test Plan

  • uv run ruff check openadapt_ml/ passes
  • uv run ruff format --check openadapt_ml/ passes
  • No changes to cli.py (critical path preserved)

🤖 Generated with Claude Code

- Move warnings.warn() after imports to fix E402 in viewer files
- Remove unused imports (Any, base64, os, Service) to fix F401
- Remove f-string without placeholders to fix F541
- Apply ruff formatting to 5 files

Files changed (7):
- benchmarks/viewer.py - E402 fix
- benchmarks/waa_deploy/api_agent.py - F401 + format
- benchmarks/azure_ops_tracker.py - format only
- benchmarks/vm_monitor.py - format only
- cloud/local.py - format only
- scripts/capture_screenshots.py - F401, F541 + format
- training/viewer.py - E402 fix

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@abrichr abrichr merged commit af64fe3 into main Jan 27, 2026
4 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.

1 participant