ForbesLindesay / browserify-middleware

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

{cache: 'dynamic'} no longer seems to work #72

Closed ianwremmel closed 9 years ago

ianwremmel commented 9 years ago

With the following config, I get a full rebuild on every page load, even if no changes have been made:

var assert = require('assert');
assert(process.env.NODE_ENV === 'development');

app.use('/scripts/', browserify('src/app/scripts/', {
  cache: 'dynamic',
  noParse: ['jquery'],
  transform: [
    'reactify',
    'envify',
  ],
}));
scottbrady commented 9 years ago

I upgraded from 3.0.1 to 5.0.0 and also starting seeing this bug. I think the caching refactor might be the cause.

https://github.com/ForbesLindesay/browserify-middleware/commit/f3f366239f7024354d8141ebf3f9778d1ef8f78a

ianwremmel commented 9 years ago

I'm seeing the problem on 4.1.0