Skip to content

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

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

refactor: Fix redundant func import in request queue client#1788
vdusek merged 4 commits into
apify: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

@Mantisus Mantisus left a comment

Copy link
Copy Markdown
Collaborator

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!

@vdusek vdusek left a comment

Copy link
Copy Markdown
Collaborator

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

vdusek commented Mar 12, 2026

Copy link
Copy Markdown
Collaborator

@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