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

Japanese public holiday issue for 2025 #141

Open
GuySturgess opened this issue Nov 29, 2024 · 4 comments
Open

Japanese public holiday issue for 2025 #141

GuySturgess opened this issue Nov 29, 2024 · 4 comments

Comments

@GuySturgess
Copy link
Contributor

The list of Japanese public holidays (using both JapanPublicHoliday().PublicHolidays(2025) and JapanPublicHoliday().IsPublicHoliday() does not include the observed date for Greenery Day 2025, which falls on a Sunday (4th May 2025) and should be observed next year on 6th May 2025. It appears to have been “Mondayised” to the 5th May 2025, but this is Children’s Day and already a public holiday. Monday 5th May 2025 appears twice in this list.

@GuySturgess
Copy link
Contributor Author

I have a fix to this if you can please grant me access o push a new code branch for review

@martinjw
Copy link
Owner

I have a fix to this if you can please grant me access o push a new code branch for review

Just fork, fix, and create a pull request - thanks!

@GuySturgess
Copy link
Contributor Author

GuySturgess commented Nov 29, 2024 via email

@martinjw
Copy link
Owner

martinjw commented Nov 29, 2024

Fork into your own github account
image
Then you can create a pull request from your forked branch to this one.

Github docs: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork

GuySturgess added a commit to GuySturgess/Holiday that referenced this issue Nov 29, 2024
martinjw pushed a commit that referenced this issue Nov 29, 2024
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

No branches or pull requests

2 participants