Skip to content
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

Open
tinohager opened this issue Nov 26, 2017 · 11 comments
Open

Complete country China - Qingming (Tomb-Sweeping Day) #52

tinohager opened this issue Nov 26, 2017 · 11 comments

Comments

@tinohager tinohager changed the title Add country China Finish country China - Qingming (Tomb-Sweeping Day) Nov 26, 2017
@tinohager tinohager changed the title Finish country China - Qingming (Tomb-Sweeping Day) Complete country China - Qingming (Tomb-Sweeping Day) Nov 26, 2017
@rubenmonteiro
Copy link
Contributor

The nuget on my pull request my give a hand on calculating this particular holiday

@tinohager
Copy link
Member Author

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.

@rubenmonteiro
Copy link
Contributor

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.
Just trying to help

@tinohager
Copy link
Member Author

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...

@tinohager
Copy link
Member Author

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

  • You are allowed to include the source code in any product (commercial, shareware, freeware or otherwise) when your product is released in binary form.
  • You are allowed to modify the source code in any way you want except you cannot modify the copyright details at the top of each module.
  • If you want to distribute source code with your application, then you are only allowed to distribute versions released by the author. This is to maintain a single distribution point for the source code.

@rubenmonteiro
Copy link
Contributor

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.

@tinohager
Copy link
Member Author

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)

@rubenmonteiro
Copy link
Contributor

Will do

@rubenmonteiro
Copy link
Contributor

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

@tinohager
Copy link
Member Author

Great, he's already added a license information, I'll ask if he could change the license to MIT

@HayriPV
Copy link

HayriPV commented Apr 13, 2021

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants