Skip to content

Conversation

@mschfh
Copy link
Collaborator

@mschfh mschfh commented Nov 8, 2025

No description provided.

@mschfh mschfh requested a review from Copilot November 8, 2025 01:58
@mschfh mschfh self-assigned this Nov 8, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR migrates from synchronous requests library to async aiohttp with retry capabilities, adds file size limit checking, and fixes a logging deprecation warning.

  • Replaces requests with aiohttp and aiohttp-retry for async HTTP operations
  • Implements exponential retry logic and file size limit validation (100 MB)
  • Updates logger.warn to logger.warning to address deprecation

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 6 comments.

File Description
pyproject.toml Replaces requests==2.32.5 dependency with aiohttp-retry==2.9.1
uv.lock Updates lock file to reflect dependency changes, removing requests, charset-normalizer, and urllib3, adding aiohttp-retry
update-lists.py Converts catalog fetch and file downloads to async/await, adds retry logic, implements file size checking, and fixes logging deprecation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mschfh mschfh marked this pull request as ready for review November 8, 2025 03:26
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.

2 participants