Skip to content

Add 2026 holidays to the static calculator - #6

Merged
CesarOrdazV merged 1 commit into
mainfrom
cursor/add-2026-holidays-fd0f
Aug 26, 2026
Merged

Add 2026 holidays to the static calculator#6
CesarOrdazV merged 1 commit into
mainfrom
cursor/add-2026-holidays-fd0f

Conversation

@CesarOrdazV

Copy link
Copy Markdown
Owner

Context

The previous PR (#5) targeted the old MAUI C# list. That app is gone: this repo is now a static GitHub Pages site. AGENTS.md says holidays live in HOLIDAYS in worktime.js, are year-specific, and a new year should be added (not swapped in place of the previous one).

The default start date is 1 January of the current year, so without 2026 dates the calculator would not exclude this year’s holidays.

Change

Append 2026 Mexican federal holidays (LFT) plus Holy Thursday and Good Friday, keeping the 2025 list:

Date Holiday
2026-01-01 New Year
2026-02-02 Constitution (first Monday of February)
2026-03-16 Benito Juárez (third Monday of March)
2026-04-02 Holy Thursday
2026-04-03 Good Friday
2026-05-01 Labor Day
2026-09-16 Independence
2026-11-16 Revolution (third Monday of November)
2026-12-25 Christmas

Also updates worktime.test.js, README.md, and the holiday note in index.html.

Tests

node --test worktime.test.js

17 passing (existing 2025 cases plus 2026 holiday coverage).

Open in Web Open in Cursor 

HOLIDAYS is year-specific, so 2026 dates are appended rather than
replacing 2025. Same set as before: LFT federal holidays plus Holy
Thursday and Good Friday. Tests, README, and the on-page note follow.

Co-authored-by: César Ordaz <cesar.ordazv@gmail.com>
@CesarOrdazV
CesarOrdazV marked this pull request as ready for review August 26, 2026 18:14
@CesarOrdazV
CesarOrdazV merged commit 85c95b4 into main Aug 26, 2026
2 checks passed
@CesarOrdazV
CesarOrdazV deleted the cursor/add-2026-holidays-fd0f branch August 26, 2026 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants