← Back to team overview

yellow team mailing list archive

[Merge] lp:~tveronezi/juju-gui/change-requires-param into lp:juju-gui

 

The proposal to merge lp:~tveronezi/juju-gui/change-requires-param into lp:juju-gui has been updated.

Description changed to:

Remove "requires" from modules-debug.js

The minification process does not read the "requires" parameters defined in "modules-debug.js". This parameter should be defined in each custom yui object that uses some kind of internal or external requirement. The sole use of "all-app-debug.js" is to define the fullpath of the file that defines a given module.

This patch removes all the existing "requires" properties from "modules-debug.js" and add then where they are needed.

https://codereview.appspot.com/6856070/


For more details, see:
https://code.launchpad.net/~tveronezi/juju-gui/change-requires-param/+merge/135198
-- 
https://code.launchpad.net/~tveronezi/juju-gui/change-requires-param/+merge/135198
Your team Juju GUI Hackers is requested to review the proposed merge of lp:~tveronezi/juju-gui/change-requires-param into lp:juju-gui.


References