search

jslicense / licensee.js

check dependency licenses against rules
https://www.npmjs.com/package/licensee
Apache License 2.0
185 stars 23 forks source link

Support conjunction #60

Closed brettz9 closed 4 years ago

brettz9 commented 4 years ago

I see parsed SPDX expressions with conjunction are excluded. I'd like to request support for expressions exclusively limited to or conjunctions. It would be nice to optionally support AND conjunctions as well if all of the AND items are present.

kemitchell commented 4 years ago

What semantics?

brettz9 commented 4 years ago

If OR is used, treat it as approved if at least one of the alternatives is present.

kemitchell commented 4 years ago

Sketch this in a PR, perhaps just test cases to start?

brettz9 commented 4 years ago

My apologies, I see conjunctions can be used, but it is just not for licenses whitelists. And for licenses, one can simply add additional licenses to the whitelist in that case. Thanks!