jslicense / licensee.js

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

Remove fs-access, use fs.access #83

Closed ericcornelissen closed 1 year ago

ericcornelissen commented 1 year ago

Closes #66 Relates to #42


Remove the no-longer-necessary dependency on fs-access and use the built-in fs module's access function instead.

I explicitly set the access mode to R_OK as the access check is performed in preparation for reading the file (and F_OK only checks whether the file is visible to the current user).

A note on the fs-access deprecation warning discussion: regardless of whether not it's a problem or which package should be responsible for addressing it, the facts are that:

  1. fs-access won't receive any updates so using the built-in fs module is preferable, and
  2. with the currently supported Node.js version there's no need to use fs-access.