Skip to content

Conversation

@kohankhaki
Copy link
Collaborator

@kohankhaki kohankhaki commented Jan 29, 2026

PR Type

Fix

Short Description

Fix linting and mypy errors in wikipedia scripts.

Tests Added

N/A


This change is Reviewable

- Add Optional types for nullable parameters
- Add type annotations for class attributes and local variables
- Fix BeautifulSoup Tag type narrowing with isinstance checks
- Fix generator type issues in sum() calls
- Add null checks for llm_model before async calls
- Shorten long docstrings and comments (W505)
- Use imperative mood in docstrings (D401)
- Add period to module docstring first line (D400)
- Use lowercase variable names in functions (N806)
- Remove .keys() from dict iteration (SIM118)
- Fix module docstring formatting (D205, D404)
@kohankhaki kohankhaki changed the title fix: resolve mypy type errors in wikipedia files fix: resolve linting and mypy type errors in wikipedia files Jan 29, 2026
@kohankhaki kohankhaki marked this pull request as ready for review January 29, 2026 19:31
False positives like 'nd' require ongoing config maintenance.
Copy link
Collaborator

@afkanpour afkanpour left a comment

Choose a reason for hiding this comment

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

@afkanpour reviewed 8 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @Negiiiin).

@kohankhaki kohankhaki merged commit 4a29638 into main Feb 4, 2026
2 checks passed
@kohankhaki kohankhaki deleted the fix/wikipedia-types branch February 4, 2026 02:41
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