-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Scripts: Added FortAwesome/Font-Awesome license in list of GPL 2 compatible licenses #12929
Scripts: Added FortAwesome/Font-Awesome license in list of GPL 2 compatible licenses #12929
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both of these licenses are GPL compatible. In the case of Font Awesome, the code is MIT, the icons are CC-BY 4.0 (ref).
An aside to the specific change, but if there's an established pattern of defining multilicensed projects using |
Yah, The two major factors for doing it in our own script, instead of using an existing package, were:
We'd probably need to keep our extractor, but something like this could work:
That at least avoids us having to write a proper expression parser. |
ea7e6bb
to
6fb6254
Compare
Let's get this in as-is and iterate on improvements separately. |
Description
I've added the license
(CC-BY-4.0 AND MIT)
in the list of GPL 2 compatible licenses, it's the license label used by https://github.com/FortAwesome/Font-Awesome FontAwesome isn't used in Gutenberg, but the addition is for other projects that can usecheck-licenses
How has this been tested?
Tests ran:
npm run check-licenses
andnpm run lint-js
Types of changes
New feature: Added
(CC-BY-4.0 AND MIT)
in the list of GPL 2 compatible licensesChecklist: