less 4.1.3 → 4.2.1

package/.eslintrc.js

@@ -0,0 +1,63 @@
+module.exports = {
+    'parser': '@typescript-eslint/parser',
+    'extends': 'eslint:recommended',
+    'parserOptions': {
+        'ecmaVersion': 2018,
+        'sourceType': 'module'
+    },
+    'plugins': ['@typescript-eslint'],
+    'env': {
+        'browser': true,
+        'node': true,
+        'mocha': true
+    },
+    'globals': {},
+    'rules': {
+        indent: ['error', 4, {
+            SwitchCase: 1
+        }],
+        'no-empty': ['error', { 'allowEmptyCatch': true }],
+        quotes: ['error', 'single', {
+            avoidEscape: true
+        }],
+        /**
+         * The codebase uses some while(true) statements.
+         * Refactor to remove this rule.
+         */
+        'no-constant-condition': 0,
+        /**
+         * Less combines assignments with conditionals sometimes
+         */
+        'no-cond-assign': 0,
+        /**
+         * @todo - remove when some kind of code style (XO?) is added
+         */
+        'no-multiple-empty-lines': 'error'
+    },
+    'overrides': [
+        {
+            files: ['*.ts'],
+            extends: ['plugin:@typescript-eslint/recommended'],
+            rules: {
+                /**
+                 * Suppress until Less has better-defined types
+                 * @see https://github.com/less/less.js/discussions/3786
+                 */
+                '@typescript-eslint/no-explicit-any': 0
+            }
+        },
+        {
+            files: ['test/**/*.{js,ts}', 'benchmark/index.js'],
+            /**
+             * @todo - fix later
+             */
+            rules: {
+                'no-undef': 0,
+                'no-useless-escape': 0,
+                'no-unused-vars': 0,
+                'no-redeclare': 0,
+                '@typescript-eslint/no-unused-vars': 0
+            }
+        },
+    ]
+}

package/Gruntfile.js

@@ -183,12 +183,6 @@ module.exports = function(grunt) {
     // Make the SauceLabs jobs
     ["all"].concat(browserTests).map(makeJob);
 
-    var path = require('path');
-
-    // Handle async / await in Rollup build for tests
-    const tsNodeRuntime = path.resolve(path.join('node_modules', '.bin', 'ts-node'));
-    const crossEnv = path.resolve(path.join('node_modules', '.bin', 'cross-env'));
-
     // Project configuration.
     grunt.initConfig({
         shell: {
@@ -222,12 +216,7 @@ module.exports = function(grunt) {
                 command: "node build/rollup.js --browser --out=./tmp/browser/less.min.js"
             },
             test: {
-                command: [
-                    // https://github.com/TypeStrong/ts-node/issues/693#issuecomment-848907036
-                    crossEnv + " TS_NODE_SCOPE=true",
-                    tsNodeRuntime + " test/test-es6.ts",
-                    "node test/index.js"
-                ].join(' && ')
+                command: 'ts-node test/test-es6.ts && node test/index.js'
             },
             generatebrowser: {
                 command: 'node test/browser/generator/generate.js'
@@ -283,7 +272,7 @@ module.exports = function(grunt) {
                 "!test/less/errors/plugin/plugin-error.js"
             ],
             options: {
-                configFile: ".eslintrc.json",
+                configFile: ".eslintrc.js",
                 fix: true
             }
         },
@@ -291,7 +280,8 @@ module.exports = function(grunt) {
         connect: {
             server: {
                 options: {
-                    port: 8081
+                    port: 8081,
+                    base: '../..'
                 }
             }
         },