Fix year_low/year_high type annotations from int to Optional[int]

[chengjilai]

Summary

Fix outdated type annotations for year_low and year_high parameters in _Scholarly.search_pubs and _Scholarly._construct_url. These parameters default to None and handle None values internally (the code explicitly checks if year_low is not None), but were annotated as int instead of Optional[int].

This causes type checkers (ty, mypy, pyright, etc.) to incorrectly report errors when callers pass None or int | None values.

Changes

• Changed year_low: int = None → year_low: Optional[int] = None
• Changed year_high: int = None → year_high: Optional[int] = None
• Added Optional to the typing imports
• Both search_pubs and _construct_url have been updated

Closes #584

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates type hints to explicitly allow None for year range parameters in scholarly search URL construction and publication searching.

Changes:

• Import Optional from typing.
• Update year_low/year_high annotations from int to Optional[int] in search_pubs.
• Update year_low/year_high annotations from int to Optional[int] in _construct_url.

---

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