-
-
Notifications
You must be signed in to change notification settings - Fork 182
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
Complete country China - Qingming (Tomb-Sweeping Day) #52
Comments
The nuget on my pull request my give a hand on calculating this particular holiday |
The package already has nearly 30.000 downloads, we have to make sure that all dependencies are license technically correct. Even new packages with 0 downloads are not very trustworthy. I think it's very nice when we move on. I would prefer if we make a separate pull request for each country we add, so I could better accept individual changes. A pull request with so many changes makes it very difficult. |
https://github.com/rubenmonteiro/AASharp The fork from the original project and I had to create a nuget because there wasn't any, so it is natural that there are 0 downloads, I made the nuget today. Why try to reinvent the wheel if it is already invented. And the project does all the calculations with no need of searching for formulas and so forth. |
Now two packages available on nuget https://www.nuget.org/packages?q=AASharp I'm happy about help! I hope we can add many more countries... |
I think this project is not only released in binary form so we must check the licence exactly http://www.naughter.com/aa.html Copyright
|
Sure, but when I forked the project, this sunday, there wasn't any nuget package, now the original github repository that I forked also as a nuget, the readme file indicates an update 4 hours ago that include the link to that nuget https://www.nuget.org/packages/AASharp So I suppose that it is better to use that nuget instead of mine because it will be updated more often then mine. |
yes I think so too, can you contact the owner of the original project (www.naughter.com/contact.html) if he could possibly change the license to MIT (https://en.wikipedia.org/wiki/MIT_License) |
Will do |
I have contact the owner of the original project and he told me to with the owner of the project that I forked https://github.com/jsauve/AASharp |
Great, he's already added a license information, I'll ask if he could change the license to MIT |
Does this help you guys? https://github.com/commenthol/date-chinese |
Source:
https://en.wikipedia.org/wiki/Public_holidays_in_China
Challenge:
5th solar term calculation (15°)
More Informations
stackoverflow.com question
The text was updated successfully, but these errors were encountered: