Issue 6488 - Dependency update fails due to missing dependencies

README.md

 Shared Adblock Plus UI code
 ===========================
 
 The user interface elements defined in this repository will be used by various
 Adblock Plus products like Adblock Plus for Firefox. Their functionality can be
 tested within this repository, even though they might not work exactly the same
 as they will do in the final product.
 
 Installing dependencies
 -----------------------
 
 Both [python 2](https://www.python.org/downloads/) and [node](https://nodejs.org/en/), as well as [npm](https://www.npmjs.com), are required to contribute to this repository.
 
 If you are installing `node` in ArchLinux, please remember to install `npm` too.
 
 After cloning this repository, enter into its folder and run: `npm install`.
 
 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 unexpected behavior that
devDependencies are not included in the production builds.

[a.giammarchi, 2018/03/15 22:36:52]

devDependencies are *never* shipped to production. These are meant to be kept
locally or for CI/testing.

In our case UI is bundled by third parts and they are responsible to pick the
right files so we're good with dependencies, simply because we are not directly
hooked in their bundling chain.

As result, I think this is OK now.

+
 Directory structure
 -------------------
 
 * Top-level files:
   * `firstRun.html` and `firstRun.js`: First-run page, see below
   * `i18n.js`: Localization functions, should be included by all pages.
   * `messageResponder.js`: Script to be used on the background page to respond
     to messages sent by UI code.
   * `background.html`, `background.js`: Test implementation of the background
     page, should *not be imported*.
@@ skipping 174 matching lines @@
   errors when adding new filters on the options page
 * `blockedURLs`: a comma-separated list of URLs that should be considered
   blocked (necessary to test the check for blocked scripts in sharing buttons).
 * `downloadStatus`: sets downloadStatus parameter for filter lists, can be used
   to trigger various filter list download errors
 * `platform=chromium`: shows the opt-out for the developer tools panel
 * `showNotificationUI=true`: simulates user having opted-out of notifications
 
 
 [crowdin]: https://crowdin.com