Comments (5)
Thanks for reporting this (so embarrassingly long ago). I've picked up work on the gem again and this definitely will be addressed.
:ca
is included in us.rb
because US holidays currently draw in the informal North American holidays, which apply to Canada, too. It's a bit of circular logic that definitely doesn't make any sense.
I'm clearing out the low-hanging fruit of definition updates before taking a proper look at these more substantial problems like these in the next few weeks (or sooner).
from holidays.
Also of note, d75aca8 will make the UnknownRegionError
much less likely. It was way over-keen before.
from holidays.
Small update, I am in the process of refactoring the codebase so that we don't have a giant single file that contains all of our logic. I'm hoping that one of two things happens during this process:
- I figure out what is causing problems like this while refactoring
- Once the codebase is split up it will be easier to diagnose problems like this.
from holidays.
I've merged in v4.0.0, which is a large refactor. This is my next target since it is now the oldest bug that I have inherited.
from holidays.
Okay! I spent some time looking at this and testing locally and as far as I can tell the core problem has been resolved. The code that handles loading of regions and definitions has changed significantly. I am going to go ahead and close out this issue. Please open a new issue if the problem (or a similar one) persists.
As an aside, once I work through the current crop of issues I am going to A) spend some time with internal refactoring for clarity and B) look at the north american informal holidays. As Alex mentioned it is a bit confusing currently.
from holidays.
Related Issues (20)
- DatesDriverBuilder does not work properly with Lunar calendar
- 4 day UK Bank Holiday in 2022 HOT 4
- Release a new version to allow this gem to be used with Ruby 3.0 HOT 4
- Get a list of all holidays possible for a given region
- Manage Belgium parent region HOT 1
- I have an issue with columbus day HOT 1
- Abuse - very important - iso8583 HOT 1
- Version Bump Request HOT 4
- Korea's new year get next year's new year
- Islamic holidays
- Día de los Muertos is not listed
- How to prevent stacked observed holidays HOT 1
- GB Substitute Days over Christmas and New Year HOT 1
- Liberation Day, Region NL
- Queen Elizabeth II passing Bank Holiday HOT 10
- Loading custom holidays clears all the provided definitions? HOT 1
- Strange behavoir with end_of_month HOT 2
- Feast of San Giusto for Trieste, Italy should be on 3rd November
- Shouldn't July 3, 2026 be "Independence Day (observed)" in the US? HOT 2
- Project updates HOT 10
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from holidays.