-
-
Notifications
You must be signed in to change notification settings - Fork 2k
Replace country-regex Part 2 #7026
Copy link
Copy link
Open
Labels
P2considered for next cycleconsidered for next cyclebugsomething brokensomething brokeninfrastructurebuild process etc.build process etc.
Milestone
Metadata
Metadata
Assignees
Labels
P2considered for next cycleconsidered for next cyclebugsomething brokensomething brokeninfrastructurebuild process etc.build process etc.
Type
Fields
Give feedbackNo fields configured for issues without a type.
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