Merge pull request #5 from mgol/get-error-hook

Define jQuery.Deferred.getErrorHook

Author: dmethvin

.eslintrc

@@ -0,0 +1,27 @@
+{
+	"root": true,
+
+	"extends": "jquery",
+
+	"reportUnusedDisableDirectives": true,
+
+	"parserOptions": {
+		"ecmaVersion": 5
+	},
+
+	// The browser env is not enabled on purpose so that code takes
+	// all browser-only globals from window instead of assuming
+	// they're available as globals. This makes it possible to use
+	// jQuery with tools like jsdom which provide a custom window
+	// implementation.
+	"env": {},
+
+	"globals": {
+		"window": true
+	},
+
+	"rules": {
+		"one-var": [ "error", { "var": "always" } ],
+		"strict": [ "error", "function" ]
+	}
+}

.eslintrc-browser.json

@@ -0,0 +1,24 @@
+{
+	"root": true,
+
+	"extends": "jquery",
+
+	"reportUnusedDisableDirectives": true,
+
+	"parserOptions": {
+		"ecmaVersion": 2022
+	},
+
+	"env": {
+		"es6": true,
+		"node": true
+	},
+	
+	"globals": {
+		"globalThis": false
+	},
+
+	"rules": {
+		"strict": [ "error", "global" ]
+	}
+}

.eslintrc-node.json

@@ -0,0 +1,38 @@
+{
+	"root": true,
+
+	"overrides": [
+		{
+			"files": "src/**/*.js",
+			"extends": "./.eslintrc-browser.json",
+			"globals": {
+				"define": false,
+				"module": false,
+				"require": false,
+				"jQuery": false
+			}
+		},
+		
+		{
+			"files": "test/node-setup.js",
+			"extends": "./.eslintrc-node.json"
+		},
+
+		{
+			"files": "test/test.js",
+			"extends": "./.eslintrc-browser.json",
+			"env": {
+				"browser": false,
+				"node": false
+			},
+			"globals": {
+				"globalThis": false,
+				"QUnit": false,
+				"jQuery": false
+			},
+			"rules": {
+				"strict": [ "error", "global" ]
+			}
+		}
+	]
+}

.eslintrc.json

@@ -0,0 +1,26 @@
+name: CI
+
+on: [push, pull_request]
+
+permissions:
+  contents: read # to fetch code (actions/checkout)
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+    steps:
+    - name: Checkout
+      uses: actions/checkout@v3
+
+    - name: Use Node.js ${{ matrix.NODE_VERSION }}
+      uses: actions/setup-node@v3.6.0
+      with:
+        node-version: 18.x
+
+    - name: Install dependencies
+      run: npm install
+
+    - name: Run tests
+      run: npm test

.github/workflows/node.js.yml

@@ -1,6 +1,6 @@
 # jquery-deferred-reporter
 
-This jQuery plugin extends the `jQuery.Deferred` features added in version 3.0 to include a stack trace whenever a Deferred throws an exception. This makes it easier to find programming errors that occur inside Deferreds. 
+This jQuery plugin extends the `jQuery.Deferred` features added in version 3.0 to include an error captured before the async barrier whenever a Deferred throws an exception. This makes it easier to find programming errors that occur inside Deferreds. 
 
 ## Why does this plugin exist?
 
@@ -10,6 +10,4 @@ The native `Promise` object as implemented in the browser tracks Promise rejecti
 
 ## Why not just put this in jQuery?
 
- Since it has to save the stack trace regardless of whether an exception will happen or not, adding this plugin makes `jQuery.Deferred` [significantly slower](https://jsfiddle.net/h20r0e6z/5/), by roughly a factor of two.
- 
-
+ Since it has to save the error regardless of whether an exception will happen or not, adding this plugin makes `jQuery.Deferred` [significantly slower](https://jsfiddle.net/h20r0e6z/5/), by roughly a factor of two.