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

Update package.json license to MIT #74

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Update package.json license to MIT #74

wants to merge 1 commit into from

Conversation

srl295
Copy link
Member

@srl295 srl295 commented Jun 7, 2022

Fixes: #73

is "license": "MIT" correct for Node.js licensed code? I assume so

@srl295 srl295 self-assigned this Jun 7, 2022
@mhdawson
Copy link
Member

I see there MIT is what is in the LICENSE file but we also have the unicode-licence.txt. From https://github.com/nodejs/full-icu-npm#license its not clear if some of the files in the repo are under the unicode licence or not. I look a bit but did not find any. @srl295 could you clarify if there are files under the unidoce licence in the repo? If not I think this update is good and matches the LICENSE file but maybe we want to clarify in license section of the readme. If so then if we can we should probably list both licences in the package.json.

@srl295
Copy link
Member Author

srl295 commented Jun 14, 2022

I see there MIT is what is in the LICENSE file but we also have the unicode-licence.txt. From https://github.com/nodejs/full-icu-npm#license its not clear if some of the files in the repo are under the unicode licence or not. I look a bit but did not find any. @srl295 could you clarify if there are files under the unidoce licence in the repo? If not I think this update is good and matches the LICENSE file but maybe we want to clarify in license section of the readme. If so then if we can we should probably list both licences in the package.json.

The icu4c data files (separate package) are under the unicode license and so marked.

This (full-icu-npm) repo came from ICU, so it's an "intake" from Unicode and thus previously under the Unicode license. Any new work since then can be contributed under the Node license. How do we want to handle that? does that help?

@mhdawson
Copy link
Member

@srl295 If all of the files were covered by the unicode licence when it came in then I think they should likely just stay that way. In that case I think we should fix the LICENSE file instead of the package.json

@github-actions
Copy link

This PR seems to be stale

@AugustinMauroy
Copy link

Hey folks ! Any news on this ?

@srl295
Copy link
Member Author

srl295 commented Apr 5, 2023

@AugustinMauroy i'll take a look.

@srl295 srl295 marked this pull request as draft April 5, 2023 15:49
@nschonni nschonni changed the title Update package.json Update package.json license to MIT Apr 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

spdx license wrong
3 participants