broccoli - npm Package Compare versions

Comparing version 0.16.2 to 0.16.3

CHANGELOG.md

 # master
 
+# 0.16.3
+
+* Do not silently swallow errors in change/error event handlers
+
 # 0.16.2
@@ -4,0 +8,0 @@

lib/watcher.js

@@ -53,4 +53,4 @@ var EventEmitter = require('events').EventEmitter
         this.emit('change', hash)
-      }.bind(this), function(error) {
-        this.emit('error', error)
+      }.bind(this), function(err) {
+        this.emit('error', err)
         if (Object.keys(self.watchedDirs).length === 0) {
@@ -62,3 +62,9 @@ console.error('An error occured before the first directory could be watched.')
       }.bind(this))
-      .then(this.check.bind(this))
+      .then(this.check.bind(this), function(err) {
+        // Errors here are usually due to errors in change/error event handlers
+        console.error('An unexpected error occurred. Watcher quitting.')
+        console.error(err.stack)
+        // Rethrow in case a debugging tool wants to catch it
+        throw err
+      })
     } else {
@@ -65,0 +71,0 @@ setTimeout(this.check.bind(this), interval)

package.json

 {
   "name": "broccoli",
   "description": "Fast client-side asset builder",
-  "version": "0.16.2",
+  "version": "0.16.3",
   "author": "Jo Liss <[e-mail]>",
@@ -6,0 +6,0 @@ "main": "lib/index.js",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

60864

increased by0.63%

Lines of code

554

increased by1.09%

No dependency changes