cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From stevengill <>
Subject [GitHub] cordova-lib pull request: CB-9436 Removes `require-tr` bundle tran...
Date Tue, 04 Aug 2015 01:45:01 GMT
Github user stevengill commented on the pull request:
    Vladimir! This is a much needed refactor and it looks great! Thank you! The previous spaghetti
code was becoming hard to manage and understand. 
    Things I like:
    * Using `cordova_plugins.js` in the browserify build and removing `cordova_requires`
    * Adding the module name field in `cordova_plugins.js` for each module.
    * Using browserify expose to map `cordova/plugin_list` to `cordova_plugins.js`. (Didn't
even know about expose)
    * Great find with aliasify! Much cleaner way to handle the bad legacy requires in plugins
compared to using requiresTr.
    Quick Question without digging to deep:
    * Wondering why you had to add require('codrdova/init') to the bundle?
How was that getting added previously? 
    I think you fixed all of the remaining bugs with your changes. @surajpindoria will continue
to test on all of the devices we have here to see if we can find bugs.
    I am going to dive deeper into your cordova.js changes now.

If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message