search

mikew / browserify-brunch

Brunch + Browserify
10 stars 7 forks source link

Used npm-check-updates to update dependencies #7

Closed sanderboom closed 7 years ago

sanderboom commented 9 years ago

Warning: atm didn't have time to test.

mrluc commented 8 years ago

I tried referencing your update-deps branch and I ran into this error:

/Users/mrluc/projects/mine/elixir/flashbang/node_modules/browserify-brunch/node_modules/watchify/index.js:14
    var cache = b._options.cache;
                          ^
TypeError: Cannot read property 'cache' of undefined
  at watchify (/Users/mrluc/projects/mine/elixir/flashbang/node_modules/browserify-brunch/node_modules/watchify/index.js:14:27)
  at new BrowserifyInstance (/Users/mrluc/projects/mine/elixir/flashbang/node_modules/browserify-brunch/lib/browserify-instance.js:21:16)
  at BrowserifyBrunch.module.exports.BrowserifyBrunch.__initInstances (/Users/mrluc/projects/mine/elixir/flashbang/node_modules/browserify-brunch/lib/index.js:95:18)
  at new BrowserifyBrunch (/Users/mrluc/projects/mine/elixir/flashbang/node_modules/browserify-brunch/lib/index.js:47:10)
  at /Users/mrluc/projects/mine/elixir/flashbang/node_modules/brunch/lib/watch.js:403:16
  at Array.map (native)
  at getPlugins (/Users/mrluc/projects/mine/elixir/flashbang/node_modules/brunch/lib/watch.js:401:6)
  at /Users/mrluc/projects/mine/elixir/flashbang/node_modules/brunch/lib/watch.js:482:18
  at /Users/mrluc/projects/mine/elixir/flashbang/node_modules/brunch/lib/helpers.js:597:12
  at /Users/mrluc/projects/mine/elixir/flashbang/node_modules/brunch/lib/helpers.js:558:16
  at /Users/mrluc/projects/mine/elixir/flashbang/node_modules/brunch/lib/helpers.js:507:12
  at /Users/mrluc/projects/mine/elixir/flashbang/node_modules/brunch/node_modules/read-components/index.js:263:16
  at /Users/mrluc/projects/mine/elixir/flashbang/node_modules/brunch/node_modules/read-components/index.js:72:14
  at FSReqWrap.cb [as oncomplete] (fs.js:228:19)
srgpqt commented 8 years ago

This pull request updates the package versions but their API is incompatible. A proper dependency update was recently merged: #9 This pull request can be closed.