fix(lint): apply ruff lint autofixes (50 fixes)#1272
Open
manderson240 wants to merge 1 commit intogoogle:mainfrom
Open
fix(lint): apply ruff lint autofixes (50 fixes)#1272manderson240 wants to merge 1 commit intogoogle:mainfrom
manderson240 wants to merge 1 commit intogoogle:mainfrom
Conversation
Applied via cohezion polish-campaign (Wave 5C / sigma-lint-autofix). Patch source: synthetic-sniffing-panda research worktree Files modified: 48 (Python sources + tests + samples) Ruff errors: 196 -> 146 (50 fixes) Tests: 204 passed, 1 skipped, 0 failures (uv run pytest tests/) The patch contains pure ruff autofixes: import sorting, removing unused imports, quote consistency, and similar low-risk transformations. No behavioral changes; full test suite passes unchanged. Patched files include a2ui core parser/schema/template modules, ADK a2a_extension, samples (component_gallery, dynamic_form_filling, todo, weather), and corresponding tests. Co-Authored-By: Claude <noreply@anthropic.com>
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
This pull request focuses on cleaning up the codebase by removing unused imports and dependencies across the Python SDK, tests, and sample applications. It also includes minor refactoring of import blocks and the conversion of f-strings to standard strings where variable interpolation was not utilized. I have no feedback to provide.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Applies ruff lint autofixes across the Python agent SDK and samples. Pure mechanical transformations: import sorting, unused-import removal, quote consistency, etc. No behavioral changes.
Verification
uv run pytest tests/)agent_sdks/python/src,agent_sdks/python/tests,samples/agent/adk/*)Origin
Generated as part of the Cohezion polish-campaign-orchestrator (Wave 5C / sigma-lint-autofix track) — a multi-wave parallel-agent code quality campaign across sibling repositories. The patch was produced by running
ruff check --fixagainst an audit ofgoogle/A2UI@mainand round-tripped here for review.Happy to drop / split / rebase if there are objections to any of the autofixes.