Update Australia holidays: change ANZAC Day holiday in 2026 (ACT)#3395
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
WalkthroughUpdated ACT ANZAC Day logic: keep the base ANZAC Day on 2026-04-25 and add a separate observed holiday on 2026-04-27; adjusted inline ACT reference link. Tests updated to assert both original and observed dates and to adjust 2027 assertions accordingly. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
|
News article that mentions the change: https://www.9news.com.au/national/anzac-day-public-holiday-act-2026-announced-to-align-with-nsw/5ab9b03e-b3c1-4b40-a940-bb3c3d3e8d00 ACT government website with the change: https://www.act.gov.au/living-in-the-act/public-holidays-school-terms-and-daylight-saving However, when I try and save the page in the WBM bot checks get in the way: https://web.archive.org/web/20260324122019/https://www.act.gov.au/web/20260324122019/https://www.act.gov.au/living-in-the-act/public-holidays-school-terms-and-daylight-saving Open to idea on how to work around this. |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@tests/countries/test_australia.py`:
- Around line 151-156: Add a one-off boundary assertion for 2027 to ensure the
ACT observed behavior is limited to 2026: after the existing 2026 assertions
that use assertHolidayName, add assertions that the regular holiday still
appears for "2027-04-25" (using assertHolidayName with the same name) and that
the observed variant (name + " (observed)") does not appear on "2027-04-27" (use
the test helper that asserts absence, e.g., assertNoHoliday or equivalent) so
accidental broadening beyond 2026 is caught.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 3adb5dc8-1499-47b9-8eb7-2ab1fe887fb4
📒 Files selected for processing (2)
holidays/countries/australia.pytests/countries/test_australia.py
dfba0f3 to
bbcf5b6
Compare
The secondary site we use for this is usually If that fails but PDF version exists i.e. the Philippines government website, we normally download those then upload them as documents to the Internet Archive directly In worst case scenario though, you can put the raw, unarchived link as-is for now and we can figure out how to deal with that later |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## dev #3395 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 312 312
Lines 18644 18644
Branches 2383 2383
=========================================
Hits 18644 18644 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
c22ec6a to
dbe6132
Compare
Co-authored-by: ~Jhellico <KJhellico@users.noreply.github.com> Signed-off-by: Panpakorn Siripanich <19505219+PPsyrius@users.noreply.github.com>
|
I'm applying code review suggestions directly for now so that we can get this PR merged in time for the first release for April 🙏 |
Co-authored-by: ~Jhellico <KJhellico@users.noreply.github.com> Signed-off-by: Panpakorn Siripanich <19505219+PPsyrius@users.noreply.github.com>
|
arkid15r
left a comment
There was a problem hiding this comment.
Thanks for updating that @simongreen-net 👍
Proposed change
The ACT government have changed the ANZAC Day holiday (again!) in 2026 to match NSW for 2026 only.
Type of change
holidaysfunctionality in general)Checklist
make checklocally; all checks and tests passed.