Add Apache 2.0 LICENSE#365
Conversation
| @@ -0,0 +1,201 @@ | |||
| Apache License | |||
There was a problem hiding this comment.
Thanks for submitting a pull request!
I am not sure if we want this, since it suggests that the kernels themselves are under this license. Some tools might automatically extract the top-level license and then assume that the code is under it.
For kernels we have the new license build.toml option. I think the build.toml of each kernel should just be under the same license as the individual kernel.
There was a problem hiding this comment.
Agreed, each kernel should probably be handled independently I think
| @@ -0,0 +1,201 @@ | |||
| Apache License | |||
There was a problem hiding this comment.
Agreed, each kernel should probably be handled independently I think
|
Thanks again for the PR! Closing this, since we now have a license attribute on each kernel. This will also be rendered on the Hub. |
3 participants
https://github.com/huggingface/kernels already uses Apache 2.0 so I assume this is an appropriate choice?