Skip to content

fix import in __init__#208

Merged
rtuszik merged 1 commit intomainfrom
fix-import
Dec 27, 2025
Merged

fix import in __init__#208
rtuszik merged 1 commit intomainfrom
fix-import

Conversation

@rtuszik
Copy link
Owner

@rtuszik rtuszik commented Dec 27, 2025

No description provided.

Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey - I've left some high level feedback:

  • Using an absolute import (src.downloader) inside src/__init__.py hardcodes the package name and can break when the project is installed under a different name; consider keeping the relative import (.downloader) or using a top-level package name that matches the installable distribution.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- Using an absolute import (`src.downloader`) inside `src/__init__.py` hardcodes the package name and can break when the project is installed under a different name; consider keeping the relative import (`.downloader`) or using a top-level package name that matches the installable distribution.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@rtuszik rtuszik merged commit 19f30ca into main Dec 27, 2025
8 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.

1 participant