ForbesLindesay / browserify-middleware

express middleware for browserify, done right
http://browserify.org
MIT License
381 stars 66 forks source link

compile.js console.dir calls pollute stdout #68

Closed focusaurus closed 9 years ago

focusaurus commented 9 years ago

https://github.com/ForbesLindesay/browserify-middleware/blob/2f1836a35948b4d6c437ae0edad4e69c0c498b49/lib/compile.js#L75

These statements are showing up on my console for the configuration below. If these are not otherwise important, could you just remove them or comment them out?

var bmwOptions = null;

if (!config.browserifyDebug) {
  bmwOptions = {
    cache: 1000 * 60 * 60 * 24 * 7, //1 week in milliseconds
    //don't mangle due to angularjs Function.toString() dependency injection
    minify: {mangle: false}
  };
}

function setup(app) {
  app.get("/browser.js", bmw([
    {"app/browser/navigation": {"entry": true}},
    "app/browser/career",
    "app/browser/home",
    "app/browser/plusParty",
    "app/browser/post",
    "app/browser/viewGallery"
  ], bmwOptions));
}
ForbesLindesay commented 9 years ago

oops, don't know how those got missed.

ForbesLindesay commented 9 years ago

These are now gone. sorry about that.