Skip to content
This repository has been archived by the owner on Aug 11, 2022. It is now read-only.

Patch bundle all dependencies #2395

Closed
wants to merge 2 commits into from

Conversation

grncdr
Copy link
Contributor

@grncdr grncdr commented Apr 24, 2012

These are the changes required in npm for npm/fstream-npm#1 to be usable.

@isaacs
Copy link
Contributor

isaacs commented Aug 17, 2012

Can you submit this to https://github.com/isaacs/read-package-json?

@anacronw
Copy link

+1, I would like this as well for npm pack. Its nice to be able to deploy an app with all its dependencies in a sandboxed environment.

Listing the dependencies twice can cause unnecessary mistakes, especially in the case where npm install --save dep is used

@timoxley
Copy link
Contributor

timoxley commented Jan 8, 2014

@grncdr bump RE #2395 (comment)

Any update on this?

@grncdr
Copy link
Contributor Author

grncdr commented Jan 8, 2014

oh hey, I don't think I ever saw that comment. FWIW I don't actually have any interest in pushing this patch forwards as I adjusted my workflows to not need it long ago. @Badunk feel free to grab those diffs and submit a new PR to read-package-json. Also, the pull request for fstream-npm referenced by this one is still open and would need to be merged for this to make any sense. (npm/fstream-npm#1)

@grncdr grncdr closed this Jan 8, 2014
@rlidwka
Copy link
Contributor

rlidwka commented Jan 24, 2014

What about this syntax?:

bundleDependencies: [
  '*',
  '!ronn',
  '!tap',
  '!npm-registry-mock',
]

Where each subsequent line overrides previous one, and "!" negates it's effect?..

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants