Skip to content
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

fix: Handle Emperor's Birthday in 2019 special case #141

Merged
merged 2 commits into from
Dec 5, 2024

Conversation

ryu0114h
Copy link
Contributor

@ryu0114h ryu0114h commented Dec 2, 2024

Fix: Handle Emperor's Birthday Correctly in 2019

This pull request addresses the issue with the handling of the Emperor's Birthday holiday in 2019. As outlined in the Japanese National Holiday, the Emperor's Birthday was not officially observed as a national holiday in 2019 due to the imperial transition.

The changes in this PR ensure that the TheEmperorsBirthday holiday is correctly handled for the year 2019, where no holiday is observed. For years prior to 2019, the holiday is set to December 23rd, and from 2020 onwards, it is set to February 23rd as per the current law.

This PR helps maintain accurate and up-to-date holiday information in the calendar library, and ensures that the 2019 special case is properly accounted for.

@ryu0114h ryu0114h marked this pull request as ready for review December 2, 2024 04:36
@ryu0114h ryu0114h marked this pull request as draft December 2, 2024 04:47
@ryu0114h ryu0114h force-pushed the bugfix/emperor-birthday-2019 branch from f936f1a to 5571ade Compare December 2, 2024 04:59
@ryu0114h ryu0114h marked this pull request as ready for review December 2, 2024 04:59
@rickar rickar merged commit 36b2290 into rickar:master Dec 5, 2024
2 checks passed
@rickar
Copy link
Owner

rickar commented Dec 5, 2024

Thank you!

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