Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: README.md

Issue 29723665: Issue 6488 - Dependency update fails due to missing dependencies (Closed)
Patch Set: Updated the README Created March 15, 2018, 8:52 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | package.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: README.md
===================================================================
--- a/README.md
+++ b/README.md
@@ -17,6 +17,8 @@
This should create and populate, both `./node_modules` folder and the `./buildtools` one.
+**Note:** [devDependencies](https://docs.npmjs.com/files/package.json#devdependencies) are not preinstalled during extension build, use [dependencies](https://docs.npmjs.com/files/package.json#dependencies) instead.
saroyanm 2018/03/15 20:57:37 Giving another thought, I'm not sure if it's an un
a.giammarchi 2018/03/15 22:36:52 devDependencies are *never* shipped to production.
+
Directory structure
-------------------
« no previous file with comments | « no previous file | package.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld