-
Notifications
You must be signed in to change notification settings - Fork 122
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
Could you please add a license for this project #193
Comments
I agree. Please follow the links to learn more about how to add a license to a repository, which licenses are available and why you should do so. Thank you very much in advance for adding a license to your project. |
This module is licenced under MIT since Feb 1 2015 as stated in the However, this change has never been published to NPM so license scanners might not pick this up. To be on the safe side, you can always fork from master and re-publish yourself. |
Thanks @MichielDeMey for pointing this out. The main problem with not publishing the license information to NPM is that automatic license checkers probably mark this package as problematic. Indeed in my company this happened which is why I added my comment above. |
Another vote for adding the license correctly. This is now wasting 15-20min of my day to manually excuse this dependency because license scanners that my organization don't pick it up easier. If it ain't on https://www.npmjs.com/package/rework then it is unlicensed. |
thanks a lot
The text was updated successfully, but these errors were encountered: