Skip to content

fix: add types-aiofiles stub to resolve mypy [import-untyped] CI failure#27

Merged
Geek-MD merged 4 commits intomainfrom
copilot/fix-blocking-call-warning
Mar 18, 2026
Merged

fix: add types-aiofiles stub to resolve mypy [import-untyped] CI failure#27
Geek-MD merged 4 commits intomainfrom
copilot/fix-blocking-call-warning

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 18, 2026

Adding aiofiles in v1.2.6 introduced a mypy [import-untyped] error in CI — distinct from [import-not-found] and not silenced by ignore_missing_imports = True in mypy.ini.

Changes

  • requirements.txt — add types-aiofiles>=23.1.0 so the CI pip install -r requirements.txt step provides the stubs before mypy runs

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: Geek-MD <25725990+Geek-MD@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix blocking call warning in advanced_downloader fix: replace blocking open() with aiofiles for async file writes (v1.2.6) Mar 18, 2026
Copilot AI requested a review from Geek-MD March 18, 2026 19:07
Copilot AI and others added 2 commits March 18, 2026 19:15
…typed error

Co-authored-by: Geek-MD <25725990+Geek-MD@users.noreply.github.com>
Co-authored-by: Geek-MD <25725990+Geek-MD@users.noreply.github.com>
Copilot AI changed the title fix: replace blocking open() with aiofiles for async file writes (v1.2.6) fix: add types-aiofiles stub to resolve mypy [import-untyped] CI failure Mar 18, 2026
@Geek-MD Geek-MD marked this pull request as ready for review March 18, 2026 19:26
@Geek-MD Geek-MD merged commit 6fd61a3 into main Mar 18, 2026
2 checks passed
@Geek-MD Geek-MD deleted the copilot/fix-blocking-call-warning branch March 18, 2026 19: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