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

CI: add cache: npm #74

Closed olleolleolle closed 2 years ago

olleolleolle commented 2 years ago

setup-node comes this functionality baked in. https://github.com/actions/setup-node#caching-packages-dependencies

ljharb commented 2 years ago

Sure, but it's best to avoid caching unless there's a reason to use it, because it reduces build determinism - why add it here?

olleolleolle commented 2 years ago

Good reasons not to, I was only going for speed. Closing.