stealjs / steal-npm

[DEPRECATED] StealJS plugin for using NPM
http://stealjs.com/
MIT License
12 stars 6 forks source link

Store the full contents of top-level package.json files? #81

Open blink1073 opened 8 years ago

blink1073 commented 8 years ago

Would you be amenable to storing the full package.json Object for the top level modules under System.npm, as opposed to the current stub? We end up re-fetching the top level package.json files to get pertinent information in our app.

justinbmeyer commented 8 years ago

Thanks for posting. I don't think we should to this by default because some package.json's can get quite large. However, I think a flag that turns this on is reasonable. Something like:

system: {
  includePackageProperties: true,
  includePackageProperties: ["property1","property2"]
}

where true would pull in everything, but you could also specify certain properties too.

blink1073 commented 8 years ago

Yep, sounds fair.

matthewp commented 8 years ago

What do you mean by "top level modules"? You mean the app's own package.json or more than that?

blink1073 commented 8 years ago

I meant the app's package.json and that of any direct dependencies.