Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Added licenses conversion for NPM packages #325

Merged
merged 1 commit into from
Jul 2, 2018

Conversation

hiqsol
Copy link
Contributor

@hiqsol hiqsol commented Jun 21, 2018

Thanks to @Roygoldman it turned out that older NPM repositories may have used objects when defining licences.

Please see more detailed explanation at hiqdev/asset-packagist#87

And example of package.json with such licenses:

https://github.com/geraintluff/tv4/blob/master/package.json#L24-L33

@richgerdes
Copy link

Thanks, @hiqsol for opening this! It will help my project greatly.

Is there anything that I can do to help move this along?

@francoispluchino francoispluchino merged commit fee88c4 into fxpio:master Jul 2, 2018
@francoispluchino
Copy link
Member

@hiqsol Thank you!

@francoispluchino francoispluchino added this to the 1.4.4 milestone Jul 2, 2018
@francoispluchino francoispluchino modified the milestones: 1.4.4, 1.4 May 4, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Development

Successfully merging this pull request may close these issues.

3 participants