Skip to content

fix: resolve ruff linting errors across agents#1218

Closed
eliasecchig wants to merge 1 commit intomainfrom
fix/ruff-linting-errors
Closed

fix: resolve ruff linting errors across agents#1218
eliasecchig wants to merge 1 commit intomainfrom
fix/ruff-linting-errors

Conversation

@eliasecchig
Copy link
Collaborator

Summary

  • Fix all ruff linting errors across multiple agent projects
  • Improve code quality and consistency

Changes

  • String split optimizations: Updated string split operations to use maxsplit parameter for better performance

    • customer-service: Fixed time range parsing
    • data-science: Fixed SQL query extraction
    • plumber-data-engineering-assistant: Fixed template path and repository parsing
    • product-catalog-ad-generation: Fixed video filename and image path parsing
  • Import organization:

    • Organized imports in retail-ai-location-strategy backend
    • Added noqa comment for necessary late import (required after sys.path modification)

Impact

  • All ruff checks now pass
  • Improved code efficiency with proper use of maxsplit in string operations
  • Better code maintainability

- Fix string split operations to use maxsplit parameter for efficiency
- Add noqa comment for necessary late import in retail-ai-location-strategy
- Organize imports in retail-ai-location-strategy backend
@eliasecchig
Copy link
Collaborator Author

Reverting linting changes - not necessary for this project

@eliasecchig eliasecchig closed this Mar 5, 2026
@eliasecchig eliasecchig deleted the fix/ruff-linting-errors branch March 5, 2026 00:12
@github-actions
Copy link

github-actions bot commented Mar 5, 2026

There is a problem with the Gemini CLI PR review. Please check the action logs for details.

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