vuejs / vue-migration-helper

CLI tool to aid in migration from Vue 1.x to 2.0
594 stars 59 forks source link

Null exception during first run #12

Closed Bodekaer closed 8 years ago

Bodekaer commented 8 years ago

Got this error when running on our project:

C:\Users\VueRocks\AppData\Roaming\npm\node_modules\vue-migration-helper\helpers\report-warning.js:10
  var library = rule.file.match(/([^\/]+)\/[^\/]+\.js/)[1]
   ^

TypeError: Cannot read property '1' of null
    at module.exports (C:\Users\VueRocks\AppData\Roaming\npm\node_modules\vue-migration-helper\helpers\report-warning.js:10:56)
    at C:\Users\VueRocks\AppData\Roaming\npm\node_modules\vue-migration-helper\helpers\check-for-deprecations.js:26:7
    at Array.some (native)
    at module.exports (C:\Users\VueRocks\AppData\Roaming\npm\node_modules\vue-migration-helper\helpers\check-for-deprecations.js:23:16)
    at Stream.<anonymous> (C:\Users\VueRocks\AppData\Roaming\npm\node_modules\vue-migration-helper\index.js:35:36)
    at emitOne (events.js:96:13)
    at Stream.emit (events.js:188:7)
    at drain (C:\Users\VueRocks\AppData\Roaming\npm\node_modules\vue-migration-helper\node_modules\through\index.js:36:16)
    at Stream.stream.queue.stream.push (C:\Users\VueRocks\AppData\Roaming\npm\node_modules\vue-migration-helper\node_modules\through\index.js:45:5)
    at emit (C:\Users\VueRocks\AppData\Roaming\npm\node_modules\vue-migration-helper\node_modules\split\index.js:37:14)
chrisvfritz commented 8 years ago

This is fixed now - see #10.

chrisvfritz commented 8 years ago

@Bodekaer It looks like there was a remaining bug that only affected Windows, but that should also be fixed now - can you confirm the latest version now works for you?

Bodekaer commented 8 years ago

Yep. It works now. Great tool! Thanks so much for building this.

All the best,

Michael Bodekaer

On 1 Oct 2016, at 13.41, Chris Fritz notifications@github.com wrote:

@Bodekaer It looks like there was a remaining bug that only affected Windows, but that should also be fixed now - can you confirm the latest version now works for you?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

chrisvfritz commented 8 years ago

@Bodekaer Excellent!