Skip to content

fix: resolve all CI ruff lint errors#31

Merged
ancongui merged 1 commit into
mainfrom
develop
Feb 22, 2026
Merged

fix: resolve all CI ruff lint errors#31
ancongui merged 1 commit into
mainfrom
develop

Conversation

@ancongui
Copy link
Copy Markdown
Contributor

Summary

  • Fix all 31 ruff lint errors that caused CI failure
  • Covers both src/ and tests/ directories

Changes

Source (N806, SIM114, UP032):

  • Rename uppercase variables in functions to lowercase across agent.py, oracle.py, settings.py, generator.py
  • Combine identical if/elif branches in oracle.py and generator.py
  • Use f-string instead of .format() in compiler.py

Tests (F401, F841, I001, E402):

  • Remove unused imports in test_pipeline_events.py, test_pipeline_integration.py, test_dynamic_model.py
  • Fix import sorting in test_project_api.py, test_dynamic_model.py, test_graphql.py
  • Add noqa for post-importorskip imports in test_graphql.py

Test plan

  • All 1,273 tests pass
  • ruff check src/ tests/ — All checks passed
  • pyright — 0 errors

- Fix N806: rename uppercase variables in functions (H_GAP→h_gap, _CHUNK_SIZE→_chunk_size, _MASK→_mask, etc.)
- Fix SIM114: combine identical if/elif branches in oracle.py and generator.py
- Fix UP032: use f-string instead of .format() in compiler.py
- Fix F401: remove unused imports (pytest, agent_registry, PipelineEngine, CompilationError) in test files
- Fix F841: remove unused variable assignment in test_pipeline_events.py
- Fix I001: sort import blocks in test files
- Fix E402: add noqa comments for post-importorskip imports in test_graphql.py
@ancongui ancongui merged commit 5836df6 into main Feb 22, 2026
12 of 13 checks passed
ancongui added a commit that referenced this pull request May 31, 2026
fix: resolve all CI ruff lint errors
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