fix: use 'allow' rather than 'white' for allowlist

jslicense / licensee.js

check dependency licenses against rules

https://www.npmjs.com/package/licensee

Apache License 2.0

185 stars 23 forks source link

fix: use 'allow' rather than 'white' for allowlist #73

Closed bnb closed 3 years ago

bnb commented 3 years ago

Switches to use "allowlist" terminology rather than "blocklist" terminology

kemitchell commented 3 years ago

@bnb do you want small things raised here in PR, or would you prefer me merge and then fettle as we like?

kemitchell commented 3 years ago

I'm thinking:

[ ] git rm the VSCode config
[ ] s/allowlisted/allowed/g
[ ] proof comments for readability

bnb commented 3 years ago

I did the first two, not sure what you mean by the last one @kemitchell 👀

kemitchell commented 3 years ago

@bnb you license your work under Apache 2.0? Any chance your employer owns this work, as done within the scope of your job?

This is mostly renaming a bunch of things, and may not fall under copyright. But this package does get used by some companies with money, so I want to make sure we stay clean on the IP side.

bnb commented 3 years ago

Wanted to give you an official answer with no question: yes, under Microsoft's Small Contribution Exception, this is unquestionably allowed. I was informed of this by Microsoft's legal organization (CELA).