Update India holidays: subdiv holidays improvement (Central and West region) - #3709
Update India holidays: subdiv holidays improvement (Central and West region)#3709ankushhKapoor wants to merge 58 commits into
Conversation
… methods chronologically
There was a problem hiding this comment.
All reported issues were addressed across 14 files (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 11 files (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 11 files (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## dev #3709 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 329 329
Lines 19752 20001 +249
Branches 2501 2502 +1
==========================================
+ Hits 19752 20001 +249 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Signed-off-by: Ankush Kapoor <work.ankushkapoor1626@gmail.com>
There was a problem hiding this comment.
1 issue found across 13 files (changes from recent commits).
Confidence score: 2/5
holidays/locale/en_US/LC_MESSAGES/IN.poremoves the Gujarat Day translation while the holiday remains defined for Gujarat, causing the May 1 holiday name to lose its intended localized text; restore the translation entry or remove/update the corresponding holiday definition consistently.
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="holidays/locale/en_US/LC_MESSAGES/IN.po">
<violation number="1" location="holidays/locale/en_US/LC_MESSAGES/IN.po:470">
P1: This diff removes the "Gujarat Day" translation entry along with the holiday itself. On dev, the Gujarat (GJ) subdivision observes May 1 as "Gujarat Day" (`self._add_holiday_may_1(tr("Gujarat Day"))` in `_populate_subdiv_gj_public_holidays`), but the PR head drops both that holiday and its IN.po entry. Gujarat is not one of the states listed in this PR (Chhattisgarh, DNH&DD, Goa, MP, Maharashtra, Rajasthan), and you noted the change was accidental during the dev-branch merge. Restore the IN.po `Gujarat Day` entry (and the `Gujarat Day` holiday in india.py)</violation>
</file>
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
| @@ -17,8 +17,8 @@ msgstr "" | |||
| "Project-Id-Version: Holidays 0.104\n" | |||
There was a problem hiding this comment.
P1: This diff removes the "Gujarat Day" translation entry along with the holiday itself. On dev, the Gujarat (GJ) subdivision observes May 1 as "Gujarat Day" (self._add_holiday_may_1(tr("Gujarat Day")) in _populate_subdiv_gj_public_holidays), but the PR head drops both that holiday and its IN.po entry. Gujarat is not one of the states listed in this PR (Chhattisgarh, DNH&DD, Goa, MP, Maharashtra, Rajasthan), and you noted the change was accidental during the dev-branch merge. Restore the IN.po Gujarat Day entry (and the Gujarat Day holiday in india.py)
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At holidays/locale/en_US/LC_MESSAGES/IN.po, line 470:
<comment>This diff removes the "Gujarat Day" translation entry along with the holiday itself. On dev, the Gujarat (GJ) subdivision observes May 1 as "Gujarat Day" (`self._add_holiday_may_1(tr("Gujarat Day"))` in `_populate_subdiv_gj_public_holidays`), but the PR head drops both that holiday and its IN.po entry. Gujarat is not one of the states listed in this PR (Chhattisgarh, DNH&DD, Goa, MP, Maharashtra, Rajasthan), and you noted the change was accidental during the dev-branch merge. Restore the IN.po `Gujarat Day` entry (and the `Gujarat Day` holiday in india.py)</comment>
<file context>
@@ -472,7 +467,7 @@ msgstr "Samvatsari Day"
#. Gujarati New Year.
msgid "Vikram Samvat New Year"
-msgstr "Vikram Samvat New Year"
+msgstr "Gujarati New Year"
#. Vassi Uttarayan.
</file context>
|
|
Single AI comment on the translation: Janjatiya Gaurav Divas: |



Proposed change
Fix subdiv holidays of India (Central and West region).
Fixes -
Chhattisgrah
Dadra and Nagar Haveli and Daman and Diu
Goa
Gujarat
Madhya Pradesh
Maharashtra
Rajasthan
Resources:
Official Notices collection
Part of GSoC and #3382
Type of change
holidaysfunctionality in general)Checklist
make checklocally; all checks and tests passed.