Skip to content

style: ruff format (#40 post-merge) - #42

Merged
johnnyhuy merged 1 commit into
mainfrom
cleanup/remove-orphaned-tls-stuff
Sep 19, 2026
Merged

johnnyhuy merged 1 commit into
mainfrom
cleanup/remove-orphaned-tls-stuff

Conversation

@johnnyhuy

Copy link
Copy Markdown
Contributor

Post-merge CI fix. The merged #40 failed ruff format --check because the local install of ruff used during development was older than the CI pin. Apply the project's format style to 4 files: collapse 3-line ternaries in chat.py / stream.py, collapse long function signatures in routing.py, and unwrap multi-line dict literals in test_cost_cap.py.

No logic changes. Tests still pass (56/56), ruff check clean.

Post-merge format cleanup. CI's pinned ruff version enforces a
tighter format than the local install; collapse 4 long-form
expressions into single lines and unwrap multi-line dict literals.

Co-authored-by: Claude <noreply@anthropic.com>
@johnnyhuy
johnnyhuy force-pushed the cleanup/remove-orphaned-tls-stuff branch from 29ecb6f to 89af929 Compare September 19, 2026 08:43
@johnnyhuy
johnnyhuy merged commit bcde5f0 into main Sep 19, 2026
12 checks passed
@johnnyhuy
johnnyhuy deleted the cleanup/remove-orphaned-tls-stuff branch September 19, 2026 08:45
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