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

Replace country-regex Part 2 #7026

Open
G-2-Z opened this issue Jun 14, 2024 · 0 comments
Open

Replace country-regex Part 2 #7026

G-2-Z opened this issue Jun 14, 2024 · 0 comments
Assignees

Comments

@G-2-Z
Copy link

G-2-Z commented Jun 14, 2024

Plotly.js still contains country-regex as a dependency. I'll just cite @keithrob from his original PR:

"The NPM package that is currently used by plotly, country-regex, is a very good package for taking common country names and correlating them to their ISO3 codes. However, it used a GPLed regex library from the R package countrycode. It is questionable whether country-regex can be a MIT license when it is a direct descendent of R version."

I'm not sure about the MIT license, but every project that uses plotly.js might be a derived work of this original GPL licensed project and therefore subject to its copyleft restrictions. This might light up in your deep inspecting license compliance tool and might make your lawyer sweaty.

see #3944
see keithrob#1

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

No branches or pull requests

2 participants