Install the Djblets-specified npm dependencies for extension packaging.

Review Request #8507 — Created Oct. 30, 2016 and submitted

Information

Djblets
release-0.10.x
c6856bb...

Reviewers

This updates the new npm installation code using during extension
packaging to instead make use of the npm dependency lists used for
Djblets itself. This ensures compatibility/consistency between Djblets
media and extension media.

The dependencies are now specified in a package.json file, which is
stored in the build/ directory. This allows for a quicker
installation/checking of npm dependencies. The resulting node_modules/
also goes in build/, in order to keep the root of the extension tree
clean.

This will allow for better maintenance of extensions and dependencies
going forward, and allow us to more easily provide the same types of
less/babel plugins we depend on ourselves with minimal maintenance.

Tested with an extension containing .less files, .js, and .es6.js.
Built the package, saw it install the modules and then compile the files.
Checked the build output and saw all the correct file contents.

Description From Last Updated

typo: optinoal

daviddavid
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        djblets/extensions/packaging.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        djblets/extensions/packaging.py
    
    
  2. 
      
david
  1. 
      
  2. djblets/extensions/packaging.py (Diff revision 1)
     
     
    Show all issues

    typo: optinoal

  3. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-0.10.x (5779aac)
Loading...