search

seatgeek / businesstime

A simple python utility for calculating business time aware timedeltas between two datetimes
BSD 2-Clause "Simplified" License
85 stars 36 forks source link

The Leap-years has not been in mind #12

Open ehooo opened 8 years ago

ehooo commented 8 years ago

Hi, i think could be a problem in the Lead-years

https://github.com/seatgeek/businesstime/blob/master/businesstime/holidays/__init__.py#L9

erwaller commented 8 years ago

Yeah, that's definitely a known issue. Fortunately, it only affects holidays defined using the -week specification (which is used for holidays like Memorial Day which are defined by their relation to the end of the month) in February. I'm not aware of any holiday that matches that description yet so I've been thinking of this as relatively low priority.