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

Docs: Write up notes for dependency management #79

Closed
coopy opened this issue Jan 25, 2016 · 1 comment
Closed

Docs: Write up notes for dependency management #79

coopy opened this issue Jan 25, 2016 · 1 comment
Assignees
@coopy
Labels
docs

Comments

@coopy
Copy link

coopy commented Jan 25, 2016

From https://formidablelabs.slack.com/archives/stack/p1453316791000597

  • Deps that are ​_imported_​ into app code need a dependency or peerDependency (probably our lodash case) or devDependency if found there.
  • Deps used ​_in_​ the archetype are in the archetype.
  • Deps in ​_both_​ ​_can_​ leverage the archetype but don't have to. But this only works for builder commands.
@coopy coopy self-assigned this Jan 25, 2016
@coopy coopy added the docs label Jan 25, 2016
@coopy
Copy link
Author

coopy commented Jan 26, 2016

Look at lodash repro again to see if NPM3 has a bug. If subdependencies are being overwritten with wrong versions due to flattening, we should open up an NPM issue.

ryan-roemer added a commit that referenced this issue Feb 5, 2016
@ryan-roemer
README.md - Add app dependencies notes, various minor fixes.
e78c1a1 
* Add section on App vs. Archetype dependencies. Fixes #79
* Add note about avoiding npm lifecycle commands. #81
* Normalize casing of headings.
* Fix code examples.
@ryan-roemer ryan-roemer mentioned this issue Feb 5, 2016
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@coopy coopy

Labels
docs
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@coopy