cburgmer
diff --git a/‎Gruntfile.js‎
Lines changed: 7 additions & 7 deletions b/‎Gruntfile.js‎
Lines changed: 7 additions & 7 deletions
diff --git a/‎eslint.config.js‎
Lines changed: 135 additions & 0 deletions b/‎eslint.config.js‎
Lines changed: 135 additions & 0 deletions
@@ -1,4 +1,3 @@
-/*global module:false*/
 module.exports = function (grunt) {
     grunt.initConfig({
         pkg: grunt.file.readJSON("package.json"),
@@ -196,20 +195,21 @@ module.exports = function (grunt) {
         },
         watch: {
             files: ["src/*.js", "test/specs/*.js"],
-            tasks: ["jshint", "jasmine"],
+            tasks: ["eslint", "jasmine"],
         },
-        jshint: {
-            all: ["src/**/*.js", "test/**/*.js", "*.js"],
+        eslint: {
             options: {
-                jshintrc: true,
+                overrideConfigFile: "eslint.config.js",
+                quiet: true,
             },
+            target: ["src/**/*.js", "test/**/*.js", "Gruntfile.js"],
         },
     });
 
     grunt.loadNpmTasks("grunt-contrib-concat");
     grunt.loadNpmTasks("grunt-contrib-connect");
     grunt.loadNpmTasks("grunt-contrib-jasmine");
-    grunt.loadNpmTasks("grunt-contrib-jshint");
+    grunt.loadNpmTasks("grunt-eslint");
     grunt.loadNpmTasks("grunt-contrib-uglify");
     grunt.loadNpmTasks("grunt-contrib-watch");
     grunt.loadNpmTasks("grunt-contrib-clean");
@@ -222,7 +222,7 @@ module.exports = function (grunt) {
         "browserify:inlineresources",
     ]);
 
-    grunt.registerTask("test", ["jshint", "connect", "jasmine"]);
+    grunt.registerTask("test", ["eslint", "connect", "jasmine"]);
 
     grunt.registerTask("build", [
         "concat:one",
 
@@ -0,0 +1,135 @@
+const globals = require("globals");
+const { defineConfig, globalIgnores } = require("eslint/config");
+
+const commonRules = {
+    // curly: true
+    curly: "error",
+    // eqeqeq: true + eqnull: true
+    eqeqeq: ["error", "always", { null: "ignore" }],
+    // immed: true (wrap immediately invoked function expressions)
+    "wrap-iife": ["error", "any"],
+    // latedef: true
+    "no-use-before-define": [
+        "error",
+        { functions: true, classes: true, variables: true },
+    ],
+    // newcap: true
+    "new-cap": "error",
+    // noarg: true
+    "no-caller": "error",
+    // undef: true
+    "no-undef": "error",
+    // trailing: true
+    "no-trailing-spaces": "error",
+    // laxbreak: true - allow line breaks before operators
+    "operator-linebreak": "off",
+};
+
+module.exports = defineConfig([
+    globalIgnores([
+        "test/performance_test_pages/**/*.js",
+        "test/fixtures/*.js",
+    ]),
+    {
+        files: ["src/**/*.js"],
+        languageOptions: {
+            ecmaVersion: 2018,
+            sourceType: "script",
+            globals: {
+                ...globals.browser,
+                util: "readonly",
+                proxies: "readonly",
+                documentUtil: "readonly",
+                documentHelper: "readonly",
+                browser: "readonly",
+                svg2image: "readonly",
+                document2svg: "readonly",
+                rasterize: "readonly",
+                // external dependencies
+                url: "readonly",
+                xmlserializer: "readonly",
+                sanedomparsererror: "readonly",
+                inlineresources: "readonly",
+            },
+        },
+        rules: {
+            ...commonRules,
+            // unused: true - but allow exported module-level vars
+            "no-unused-vars": [
+                "error",
+                {
+                    varsIgnorePattern:
+                        "^(util|proxies|documentUtil|documentHelper|browser|svg2image|document2svg|rasterize|rasterizeHTML)$",
+                },
+            ],
+            // strict: true
+            strict: ["error", "function"],
+        },
+    },
+    {
+        files: ["test/**/*.js"],
+        languageOptions: {
+            ecmaVersion: 2018,
+            sourceType: "script",
+            globals: {
+                ...globals.browser,
+                ...globals.node,
+                Promise: true,
+                // console etc from devel
+                console: true,
+                alert: true,
+                // jasmine globals
+                jasmine: true,
+                describe: true,
+                it: true,
+                xit: true,
+                beforeEach: true,
+                afterEach: true,
+                expect: true,
+                fail: true,
+                spyOn: true,
+                // project test helpers
+                ifNotInWebkitOrBlinkIt: true,
+                testHelper: true,
+                diffHelper: true,
+                // project globals
+                util: true,
+                proxies: true,
+                documentUtil: true,
+                documentHelper: true,
+                browser: true,
+                svg2image: true,
+                document2svg: true,
+                rasterize: true,
+                rasterizeHTML: true,
+                imagediff: true,
+                // external dependencies
+                inlineresources: true,
+                isEqual: true,
+            },
+        },
+        rules: {
+            ...commonRules,
+            // unused: true - allow unused function parameters (common in test callbacks)
+            "no-unused-vars": ["error", { args: "none" }],
+            // strict: true - disabled in test files as they have mixed patterns
+            strict: "off",
+        },
+    },
+    {
+        files: ["Gruntfile.js"],
+        languageOptions: {
+            ecmaVersion: 2018,
+            sourceType: "script",
+            globals: {
+                ...globals.node,
+                module: true,
+            },
+        },
+        rules: {
+            ...commonRules,
+            "no-unused-vars": "error",
+            strict: "off",
+        },
+    },
+]);