-
-
Couldn't load subscription status.
- Fork 559
Update India holidays: add Ugadi holiday for AP, KA, TS subdivisions #2953
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
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## dev #2953 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 299 299
Lines 17842 17845 +3
Branches 2295 2295
=========================================
+ Hits 17842 17845 +3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
So just use |
|
@KJhellico yeah Gudi Padwa and Ugadi always fall on the same date according to the Hindu lunisolar calendar. The only reason I added In Maharashtra, the festival is celebrated as Gudi Padwa. In Karnataka, Andhra Pradesh, and Telangana, the very same date is observed as Ugadi. So while the underlying date is the same, the holiday names differ by state. To reflect this regional naming correctly, I’ve kept them separate in code, but both functions resolve to the exact same date. [] (https://www.timeanddate.com/holidays/india/ugadi) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🇮🇳
Summary by CodeRabbit
WalkthroughAdds Ugadi (Gudi Padwa) holiday entries for Andhra Pradesh, Karnataka, and Telangana in India; updates Hindu group docstring for Gudi Padwa/Ugadi; adds/updates translations in en_IN, en_US, and hi locales; adjusts tests to expect combined “Gudi Padwa; Ugadi” naming. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested reviewers
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
🧪 Generate unit tests
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro 📒 Files selected for processing (6)
🧰 Additional context used🧠 Learnings (31)📚 Learning: 2025-03-08T11:28:48.652ZApplied to files:
📚 Learning: 2025-06-28T10:39:19.185ZApplied to files:
📚 Learning: 2025-05-06T15:25:44.333ZApplied to files:
📚 Learning: 2025-03-05T17:51:00.633ZApplied to files:
📚 Learning: 2025-06-25T10:09:29.029ZApplied to files:
📚 Learning: 2025-03-30T18:25:07.087ZApplied to files:
📚 Learning: 2025-04-17T17:08:48.082ZApplied to files:
📚 Learning: 2025-06-10T12:17:58.880ZApplied to files:
📚 Learning: 2025-03-30T13:33:31.598ZApplied to files:
📚 Learning: 2025-06-11T18:32:25.595ZApplied to files:
📚 Learning: 2025-05-06T21:07:11.577ZApplied to files:
📚 Learning: 2025-04-02T18:38:35.164ZApplied to files:
📚 Learning: 2025-05-13T13:23:11.375ZApplied to files:
📚 Learning: 2025-06-14T20:43:15.370ZApplied to files:
📚 Learning: 2025-08-19T21:00:47.849ZApplied to files:
📚 Learning: 2025-08-21T04:56:03.780ZApplied to files:
📚 Learning: 2025-04-03T16:58:27.175ZApplied to files:
📚 Learning: 2025-06-14T20:12:37.212ZApplied to files:
📚 Learning: 2025-07-14T20:23:48.198ZApplied to files:
📚 Learning: 2025-05-10T04:32:15.760ZApplied to files:
📚 Learning: 2025-08-25T09:57:22.291ZApplied to files:
📚 Learning: 2025-06-18T10:07:58.780ZApplied to files:
📚 Learning: 2025-06-19T02:34:18.382ZApplied to files:
📚 Learning: 2025-08-11T10:14:28.517ZApplied to files:
📚 Learning: 2025-06-29T09:37:35.283ZApplied to files:
📚 Learning: 2025-04-05T08:12:19.986ZApplied to files:
📚 Learning: 2025-09-17T15:53:16.940ZApplied to files:
📚 Learning: 2025-06-25T20:55:00.642ZApplied to files:
📚 Learning: 2025-04-10T10:58:12.704ZApplied to files:
📚 Learning: 2025-06-10T05:08:07.939ZApplied to files:
📚 Learning: 2025-05-10T04:02:13.815ZApplied to files:
🧬 Code graph analysis (1)holidays/countries/india.py (1)
⏰ 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 (13)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 Ruff (0.13.1)holidays/countries/india.py�[1;31mruff failed�[0m holidays/groups/hindu.py�[1;31mruff failed�[0m tests/countries/test_india.py�[1;31mruff failed�[0m Tip 👮 Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Please see the documentation for more information. Example: reviews:
pre_merge_checks:
custom_checks:
- name: "Undocumented Breaking Changes"
mode: "warning"
instructions: |
Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).Please share your feedback with us on this Discord post. 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 |
|



Proposed change
Add Ugadi holiday for Andhra Pradesh (AP), Telangana (TS), and Karnataka (KA) states #2445
hi,en_IN,en_US), andmake checkpasses.This ensures accurate state-specific holidays without duplicating dates.
Type of change
Checklist
make checklocally; all checks and tests passed.This PR solves issue #2445