Skip to content

refactor: Fix redundant func import in request queue client#1788

Merged
vdusek merged 4 commits intoapify:masterfrom
LincolnBurrows2017:fix/redundant-func-import
Apr 14, 2026
Merged

refactor: Fix redundant func import in request queue client#1788
vdusek merged 4 commits intoapify:masterfrom
LincolnBurrows2017:fix/redundant-func-import

Conversation

@LincolnBurrows2017
Copy link
Copy Markdown
Contributor

Summary

This PR fixes a redundant import in src/crawlee/storage_clients/_sql/_request_queue_client.py:

  • Removes duplicate from sqlalchemy import func as sql_func import
  • Uses consistent func naming throughout the file

This improves code clarity by eliminating the confusing duplicate import for the same module.

AI Bot added 2 commits March 11, 2026 22:45
- Remove duplicate 'from sqlalchemy import func as sql_func' import
- Use consistent 'func' naming throughout the file
@vdusek vdusek changed the title Fix redundant func import in _request_queue_client.py refactor: Fix redundant func import in request queue client Mar 12, 2026
Copy link
Copy Markdown
Collaborator

@Mantisus Mantisus left a comment

Choose a reason for hiding this comment

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

Thank you for noticing and correcting that!

Copy link
Copy Markdown
Collaborator

@vdusek vdusek left a comment

Choose a reason for hiding this comment

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

Hi @LincolnBurrows2017, thanks for your contribution! I'll just make the PR refactor type, as it is not a bug fix.

@vdusek
Copy link
Copy Markdown
Collaborator

vdusek commented Mar 12, 2026

@LincolnBurrows2017 please fix linting issues

vdusek and others added 2 commits April 14, 2026 12:59
@vdusek vdusek merged commit f241ead into apify:master Apr 14, 2026
30 checks passed
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.

4 participants