Summary by CodeRabbit

• Chores

  • Updated tooling configuration to preserve runtime typing hints during automated upgrades.

• Refactor

  • Modernized internal type-hinting and import handling across multiple country holiday modules. No changes to behavior or public APIs.

• Style

  • Minor formatting and lint adjustments to improve readability and consistency. No functional impact.

Walkthrough

Adds postponed evaluation of annotations and TYPE_CHECKING-guarded date imports across several country holiday modules; updates pre-commit pyupgrade args to keep runtime typing. taiwan.py also reformats a method signature and adds a lint suppression. No runtime behavior or public API changes.

Changes

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Update mypy configuration #2464 — Overlapping typing/import changes (includes edits to holidays/countries/taiwan.py and other type-guarded date imports).

Suggested reviewers

• PPsyrius

Pre-merge checks and finishing touches

Learnt from: KJhellico
PR: vacanza/holidays#2631
File: tests/countries/test_sint_maarten.py:62-0
Timestamp: 2025-06-14T21:12:07.224Z
Learning: KJhellico prefers to focus on completing and reviewing the main holiday implementation code before doing detailed reviews of the corresponding test files.

Learnt from: KJhellico
PR: vacanza/holidays#2833
File: tests/countries/test_uganda.py:15-0
Timestamp: 2025-08-20T19:46:15.625Z
Learning: In the holidays project, the standard import pattern for country test files is `from holidays.countries.<country> import Country, CODE1, CODE2` (direct module import), used by ~75% of country test files. Only a minority (~25%) use the aggregated public API import `from holidays.countries import Country, CODE1, CODE2`. The direct module import pattern should be used for new country test files to follow the established convention.

Learnt from: PPsyrius
PR: vacanza/holidays#2615
File: tests/countries/test_anguilla.py:20-24
Timestamp: 2025-06-16T15:48:33.931Z
Learning: The holidays project uses `make pre-commit` test which includes `ruff` as the primary linter. If `ruff` passes, then Pylint warnings that don't appear in `ruff` are considered acceptable and can be ignored.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (f322abc) to head (4f33b1b).
⚠️ Report is 3 commits behind head on dev.

@@            Coverage Diff             @@
##               dev     #2949    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files          296       299     +3     
  Lines        17652     17842   +190     
  Branches      2271      2295    +24     
==========================================
+ Hits         17652     17842   +190     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud