bundledDependencies should be honored before bundleDependencies #2833

Closed
bmeck opened this Issue Sep 26, 2012 · 3 comments

Comments

Projects
None yet
4 participants
Contributor

bmeck commented Sep 26, 2012

Mis-spelled config name should not be prioritized over correctly spelled one.

https://github.com/isaacs/npm/blob/3bf7d2f7b34a1fdc15e4e7c58360c56157da5f70/lib/install.js#L289

Contributor

luk- commented Mar 9, 2013

imo not a huge deal 🌴 but if you want to submit the PR I'll merge it

Contributor

ralt commented Mar 10, 2013

Instead of letting the issue rot (OP opened this issue 6 months ago), I thought it was better to PR right away, so that we can close this issue.

Owner

isaacs commented Mar 11, 2013

It looks like bundledDependencies is always rewritten to bundleDependencies in the read-package-json module.

The double-d is actually the typo.

isaacs closed this Mar 11, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment