-
-
Notifications
You must be signed in to change notification settings - Fork 554
Drop Python 3.9 support #2998
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: dev
Are you sure you want to change the base?
Drop Python 3.9 support #2998
Conversation
Summary by CodeRabbit
WalkthroughDropped Python 3.9 support and modernized many type hints to PEP 604 union syntax (A | B) across CI, tooling, core, calendars, groups, countries, scripts, and tests. Adjusted pyproject/tox/CI matrices and cleaned up unused typing imports. No runtime logic changes reported. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested reviewers
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro 📒 Files selected for processing (1)
🧰 Additional context used🧠 Learnings (1)📚 Learning: 2025-04-08T14:46:10.656Z
Applied to files:
🪛 Ruff (0.13.3)holidays/holiday_base.py205-205: Mutable class attributes should be annotated with (RUF012) 235-235: Boolean-typed positional argument in function definition (FBT001) 235-235: Boolean default positional argument in function definition (FBT002) 236-236: Boolean-typed positional argument in function definition (FBT001) 236-236: Boolean default positional argument in function definition (FBT002) 798-798: Missing type annotation for (ANN002) 957-957: Dynamically typed expressions (typing.Any) are disallowed in (ANN401) 957-957: Dynamically typed expressions (typing.Any) are disallowed in (ANN401) 1174-1174: Dynamically typed expressions (typing.Any) are disallowed in (ANN401) 1174-1174: Dynamically typed expressions (typing.Any) are disallowed in (ANN401) ⏰ Context from checks skipped due to timeout of 300000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
🔇 Additional comments (1)
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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## dev #2998 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 305 305
Lines 18039 18005 -34
Branches 2329 2329
=========================================
- Hits 18039 18005 -34 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: Panpakorn Siripanich <19505219+PPsyrius@users.noreply.github.com>
|
Proposed change
importlib-metadata
which is now built-in asimportlib.metadata
).zipp>=3.19.1
pinning indev
andtest
requirements (from what I understand, this was previously only used byimportlib-metadata
).Type of change
holidays
functionality in general)Checklist
make check
locally; all checks and tests passed.