Skip to content

Refactor Pexel Downloader: Split functionality into client, config, a…#9

Merged
Gabriellgpc merged 5 commits intomainfrom
luis/migrate-to-uv-and-typer
Mar 14, 2026
Merged

Refactor Pexel Downloader: Split functionality into client, config, a…#9
Gabriellgpc merged 5 commits intomainfrom
luis/migrate-to-uv-and-typer

Conversation

@Gabriellgpc
Copy link
Copy Markdown
Collaborator

…nd constants modules

  • Created client.py for PexelDownloader class handling image and video downloads.
  • Introduced config.py for managing configuration settings and API key retrieval.
  • Added constants.py for defining base URL and enums for image and video sizes.
  • Removed old downloader.py and pexel_downloader.py files to streamline the codebase.
  • Updated project structure with a new pyproject.toml for dependency management.
  • Implemented tests for the new structure, including CLI and downloader functionality.
  • Removed outdated requirements.txt and setup.py in favor of the new project configuration.

…nd constants modules

- Created `client.py` for PexelDownloader class handling image and video downloads.
- Introduced `config.py` for managing configuration settings and API key retrieval.
- Added `constants.py` for defining base URL and enums for image and video sizes.
- Removed old `downloader.py` and `pexel_downloader.py` files to streamline the codebase.
- Updated project structure with a new `pyproject.toml` for dependency management.
- Implemented tests for the new structure, including CLI and downloader functionality.
- Removed outdated `requirements.txt` and `setup.py` in favor of the new project configuration.
@Gabriellgpc Gabriellgpc self-assigned this Mar 14, 2026
@Gabriellgpc Gabriellgpc merged commit 583c6af into main Mar 14, 2026
4 checks passed
@Gabriellgpc Gabriellgpc deleted the luis/migrate-to-uv-and-typer branch March 14, 2026 22:30
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