Skip to content

chinese holidays#87

Open
MenoData wants to merge 1 commit intosvendiedrichsen:masterfrom
MenoData:master
Open

chinese holidays#87
MenoData wants to merge 1 commit intosvendiedrichsen:masterfrom
MenoData:master

Conversation

@MenoData
Copy link

see #43

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 83.064% when pulling 570661e on MenoData:master into e359ee2 on svendiedrichsen:master.

@svendiedrichsen
Copy link
Owner

Great work. Glad to include it. Just one minor request. Please see review.

@MenoData
Copy link
Author

Hm, I have problems to see any review or minor request. Anyway, this PR will ensure to find the right days of official chinese holidays and those of Hongkong. What is not yet tackled are the complex compensation rules when ever a holiday falls on a weekend (see wikpedia), but this might be realized in another second step (you know better your API). And sometimes the government of China can suddenly change the compensation rules as happened several times in the past, so ultimate safety for any holiday dates is only given by official announcements in December.

@maccamlcQ
Copy link
Contributor

I had a quick look at this PR, and trying to understand how the Chinese holidays are arranged. Would the intention for Chinese New Year, for example, be to list holiday as only the actual day, or try to find a way to cover the 7 day holiday from Eve to 6 days later?

I assume would need some additional constructs to calculate this type of requirement.

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.

4 participants