Skip to content

feat(python): add ruff linter and formatter#23

Draft
alexander-akhmetov wants to merge 1 commit intomainfrom
alexander-akhmetov/python-linter
Draft

feat(python): add ruff linter and formatter#23
alexander-akhmetov wants to merge 1 commit intomainfrom
alexander-akhmetov/python-linter

Conversation

@alexander-akhmetov
Copy link
Copy Markdown
Collaborator

@alexander-akhmetov alexander-akhmetov commented Apr 10, 2026

Add ruff for linting and formatting across all Python SDK packages.
Reformatted the code.


Note

Low Risk
Mostly introduces Python lint/format tooling (ruff) and applies automated reformatting/type-hint modernizations across Python modules; functional impact should be minimal aside from small refactors like attribute assignment and error-message formatting.

Overview
Adds ruff to the repo toolchain (via mise.toml) with new lint:py and format:py tasks, and wires a dedicated Python Lint job into CI to run ruff check plus ruff format --check.

Applies a broad ruff format pass and related cleanup across Python SDK, providers, frameworks, and tests (import ordering, line wrapping, and modern X | None annotations), with a few tiny refactors such as with_sigil_google_adk_plugins using direct target.plugins = ... assignment and minor error-message string construction tweaks.

Reviewed by Cursor Bugbot for commit 070d2ac. Bugbot is set up for automated code reviews on this repo. Configure here.

@alexander-akhmetov alexander-akhmetov force-pushed the alexander-akhmetov/python-linter branch from ace8c60 to 0cbd8ce Compare April 10, 2026 21:08
Add ruff for linting and formatting across all
Python SDK packages.
@alexander-akhmetov alexander-akhmetov force-pushed the alexander-akhmetov/python-linter branch from 0cbd8ce to 070d2ac Compare April 10, 2026 21:22
@alexander-akhmetov
Copy link
Copy Markdown
Collaborator Author

bugbot review

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 070d2ac. Configure here.

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