-
Notifications
You must be signed in to change notification settings - Fork 442
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
Add Bahamas holidays #1517
Add Bahamas holidays #1517
Conversation
@PPsyrius, your research is deep and thorough as always 👍. About observed holidays. Wikipedia and OfficeHolidays says that:
but I can't find any regulatory act that establishes this. On Timeanddate we can see observance of these rules only for the New Year's Day and Discovery Day, and only until 2012. |
Seems you forgot to add line to |
Co-Authored-By: ~Jhellico <KJhellico@users.noreply.github.com>
…an countries Co-authored-by: ~Jhellico <KJhellico@users.noreply.github.com>
Well, thanks to your comment, I happened to come across this site run by the Bahamas government with all published laws since 1923. Take the Public Holidays Act (as last published in 2001) or the Majority Rule (Public Holiday) Act 2013 already cited in the PR for example. Turns out Bahamas's "Labour Day" got its name changed as recently as 2013 in which the existing codes need to be adjusted. On the topic of in-lieu, after checking the 2022 calendars we have on hand, the observance for New Year's Day appears to only apply if it falls on |
Also cited official law source instead of ILO ones
Pull Request Test Coverage Report for Build 6579180463
💛 - Coveralls |
Co-authored-by: ~Jhellico <KJhellico@users.noreply.github.com>
Co-authored-by: ~Jhellico <KJhellico@users.noreply.github.com>
Co-authored-by: ~Jhellico <KJhellico@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great stuff, @PPsyrius!
The code wraps pretty much all holidays with ObservedHolidayBase::_add_observed
which looks fine but is not really needed for some cases (that never fall on SAT/SUN). I'm going to let you decide whether to keep it as is or apply the PR suggestions.
Co-authored-by: Arkadii Yakovets <ark@cho.red>
Co-authored-by: Arkadii Yakovets <ark@cho.red>
Co-authored-by: Arkadii Yakovets <ark@cho.red>
Co-authored-by: Arkadii Yakovets <ark@cho.red>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🇧🇸 LGTM
Proposed change
Add Bahamas holidays.
Closes #1152.
Type of change
Checklist
make pre-commit
command generates no changesmake test
,make tox
(we strongly encourage adding tests to your code)