-
Notifications
You must be signed in to change notification settings - Fork 210
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
add license title #65
Conversation
It's not strictly required, but it's useful metadata, and part of the recommended license template text (see http://choosealicense.com/licenses/isc/ and https://opensource.org/licenses/isc-license)
@mourner would you mind if I sent similar PRs to the remaining mapbox repos? I've already sent a bunch, but stopped because I didn't want to spam too much :) |
@waldyrious I wouldn't mind, but maybe not all Mapbox repos but the few relatively popular ones (we have about 700 open repos and that would be too much for notifications). |
@mourner no problem :) I was planning on targeting the top ones by stars, so that should amount to about 20 PRs (in addition to those I've already submitted). Is that ok? |
Yes, I think that's fine. |
cool :) I'll get started shortly. |
@mourner all done :) |
as discussed in mapbox/earcut#65
as discussed in mapbox/earcut#65
It's not strictly required, but it's useful metadata, and part of the recommended license template text (see http://choosealicense.com/licenses/isc/ and https://opensource.org/licenses/isc-license)