Use fetch package.json for deps from node_modules.

kessler / license-report

create a short report about a project's dependencies (license, url etc)

MIT License

227 stars 39 forks source link

Use fetch package.json for deps from node_modules. #4

Closed aindlq closed 6 years ago

aindlq commented 8 years ago

Hi,

That is an attempt to resolve deps package.json from local filesystem. It uses read-installed, which is library used for "npm ls" command.

do not try to guess package link, it can be very misleading
use read-installed to get package.json for all installed dependencies
remove semver validation, because read-installed handles it

kessler commented 8 years ago

Need to take a deeper look, will do, this weekend. npm is update to date with previous PR

kessler commented 8 years ago

Had a look, all and all I like it since it makes for less code, however, my use case still requires me to be able to operate in an environment where just the package.json file exists and nothing else. Maybe we can do two seperate implementation with a single interface (registry vs local disk) ?

kessler commented 7 years ago

Hi, it doesn't look like we concluded our discussion about operating using a package.json and nothing else. Any thoughts on that?