Skip to content

Update Australia holidays: change ANZAC Day holiday in 2026 (ACT)#3395

Merged
arkid15r merged 7 commits into
vacanza:devfrom
simongreen-net:act-anzac-day-2026
Mar 31, 2026
Merged

Update Australia holidays: change ANZAC Day holiday in 2026 (ACT)#3395
arkid15r merged 7 commits into
vacanza:devfrom
simongreen-net:act-anzac-day-2026

Conversation

@simongreen-net

Copy link
Copy Markdown
Contributor

Proposed change

The ACT government have changed the ANZAC Day holiday (again!) in 2026 to match NSW for 2026 only.

Type of change

  • New country/market holidays support (thank you!)
  • Supported country/market holidays update (calendar discrepancy fix, localization)
  • Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
  • Dependency update (version deprecation/pin/upgrade)
  • Bugfix (non-breaking change which fixes an issue)
  • Breaking change (a code change causing existing functionality to break)
  • New feature (new holidays functionality in general)

Checklist

@github-actions github-actions Bot added the test label Mar 24, 2026
@coderabbitai

coderabbitai Bot commented Mar 24, 2026

Copy link
Copy Markdown
Contributor

Note

Reviews paused

It 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 reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review

Walkthrough

Updated 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

Cohort / File(s) Summary
ANZAC Day Observation Logic
holidays/countries/australia.py
Changed ACT handling for ANZAC Day (2026): stop moving the base 25‑Apr holiday, add a separate observed holiday via _add_observed(...), and update the ACT reference link.
Test Assertions
tests/countries/test_australia.py
Updated ACT-specific assertions in test_anzac_day: assert ANZAC Day on 2026-04-25, assert ANZAC Day (observed) on 2026-04-27, and adjust 2027 presence/absence checks for original and observed labels.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • arkid15r
  • PPsyrius
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description check ✅ Passed The description explains the ACT government's change to ANZAC Day in 2026 and properly categorizes it as a supported holidays update.
Title check ✅ Passed The title accurately describes the main change: updating ANZAC Day holiday handling in ACT for 2026, which is the core focus of the changeset across both implementation and test files.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

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

@simongreen-net

Copy link
Copy Markdown
Contributor Author

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

📥 Commits

Reviewing files that changed from the base of the PR and between bd01922 and 13c964b.

📒 Files selected for processing (2)
  • holidays/countries/australia.py
  • tests/countries/test_australia.py

Comment thread tests/countries/test_australia.py Outdated
@PPsyrius

Copy link
Copy Markdown
Collaborator

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.

The secondary site we use for this is usually archive.todaywhich works in this case
http://archive.today/2026.03.24-132232/https://www.act.gov.au/living-in-the-act/public-holidays-school-terms-and-daylight-saving

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

codecov Bot commented Mar 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (8e468e0) to head (28bf0e8).
⚠️ Report is 1 commits behind head on dev.

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.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread tests/countries/test_australia.py Outdated
PPsyrius and others added 2 commits March 29, 2026 23:35
Co-authored-by: ~Jhellico <KJhellico@users.noreply.github.com>
Signed-off-by: Panpakorn Siripanich <19505219+PPsyrius@users.noreply.github.com>
@PPsyrius PPsyrius requested a review from KJhellico March 29, 2026 16:35
@PPsyrius

Copy link
Copy Markdown
Collaborator

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 🙏

Comment thread tests/countries/test_australia.py Outdated
Co-authored-by: ~Jhellico <KJhellico@users.noreply.github.com>
Signed-off-by: Panpakorn Siripanich <19505219+PPsyrius@users.noreply.github.com>
@PPsyrius PPsyrius requested a review from KJhellico March 29, 2026 17:02

@KJhellico KJhellico left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@KJhellico KJhellico changed the title Add changes for ANZAC Day holiday in 2026 in ACT Update Australia holidays: change ANZAC Day holiday in 2026 (ACT) Mar 29, 2026
@sonarqubecloud

Copy link
Copy Markdown

@arkid15r arkid15r left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for updating that @simongreen-net 👍

@arkid15r arkid15r added this pull request to the merge queue Mar 31, 2026
Merged via the queue into vacanza:dev with commit d8cac3a Mar 31, 2026
32 checks passed
@KJhellico KJhellico mentioned this pull request Apr 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants