Skip to content

Fix Ruff lint violations - #165

Merged
timkpaine merged 2 commits into
mainfrom
tkp/ruff
Aug 1, 2026
Merged

Fix Ruff lint violations#165
timkpaine merged 2 commits into
mainfrom
tkp/ruff

Conversation

@timkpaine

@timkpaine timkpaine commented Jul 31, 2026

Copy link
Copy Markdown
Member

Updates Python source and tests to satisfy diagnostics exposed by the current Ruff release while leaving the Ruff configuration unchanged.

  • Modernize type annotations and dictionary construction.
  • Use precise runtime exceptions and consistent integer parsing for RAYDAR_PORT.
  • Remove invalid formatter suppression comments and simplify nested conditionals.
  • Constrain FastAPI below 0.139.2 because FastAPI 0.139.2+ adds lock-bearing app state that Ray 2.56.1 cannot serialize for @serve.ingress.

Validation:

  • make lint
  • python -m build -w -n
  • Clean isolated install with FastAPI 0.139.1 and Ray 2.56.1 successfully imports and serializes the Raydar app.

make test-py was also attempted locally, but Ray Serve could not bind 127.0.0.1:8000 because an existing VS Code Code Helper process owns that port; the second test consequently reached that unrelated listener.

Signed-off-by: Tim Paine <3105306+timkpaine@users.noreply.github.com>
Signed-off-by: Tim Paine <3105306+timkpaine@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown
Contributor

Test Results

2 tests  ±0   2 ✅ ±0   48s ⏱️ -1s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 2169a15. ± Comparison against base commit e8a6338.

@timkpaine
timkpaine merged commit 889cd33 into main Aug 1, 2026
4 checks passed
@timkpaine
timkpaine deleted the tkp/ruff branch August 1, 2026 00:00
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