-
-
Notifications
You must be signed in to change notification settings - Fork 16
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
Include LICENSE file in the installed node_modules #51
Comments
License checkers should be able to look at the package.json? Do you use tools that don’t do that? |
I don’t think your solution works. If we link from |
We could either copy the license in the build scripts, or just copy the license into all packages once. We already include the license in packages in other monorepos, such as remark. |
They are able to detect the license type, but they are not able to detect the license text, which is required for creating copyrights/attribution files.
To be honest, I am not quiet sure that it would work or not. Another solution that I just bumped into in another monorepo, is just copying the license into all packages. |
Thank you a lot, another thing because I was looking also into doing the same in other repos that you are maintaining. Do you think it's good if you add the license file also to the installed packages? |
Sorry, I don’t understand your question. Can you rephrase please |
Yes sure, |
That’s what I just did? What is the question? |
My fault sorry, I didn't notice that they are already included, and the changes you did in the pipeline-package.js. |
Initial checklist
Problem
Including the license file in the installed node_modules, makes the packages compatible with license checkers and attribution file generators, that a lot of developers need for their work.
Solution
Add the license file in the package.json to the 'files' field of each package.
Alternatives
Nothing on my mind, that can replace that.
The text was updated successfully, but these errors were encountered: