Merge pull request #315 from humanmade/upgrade-eslint

Upgrade eslint and stylelint

Author: abhishek-kaushik

.nvmrc

@@ -0,0 +1 @@
+22

.travis.yml

@@ -14,32 +14,29 @@ branches:
 jobs:
   include:
     - language: php
-      php: 7.2
-      install:
-        - composer install
-      script:
-        - vendor/bin/phpunit
-    - language: php
-      php: 7.3
-      install:
-        - composer install
-      script:
-        - vendor/bin/phpunit
-    - language: php
-      php: 7.4
+      php: 8.1
       install:
-        - composer install
+        # For PHP 8.0+, we need to ignore platform reqs as PHPUnit 7 is still used.
+        - composer install --ignore-platform-reqs
       script:
-        - vendor/bin/phpunit
+        # For PHP 8.1+, we need to ignore the config file so that PHPUnit 7 doesn't try to read it and cause an error.
+        # Instead, we pass all required settings as part of the phpunit command.
+        - vendor/bin/phpunit --no-configuration --bootstrap=tests/bootstrap.php --dont-report-useless-tests tests/AllSniffs.php
+        - vendor/bin/phpunit --no-configuration --bootstrap=tests/bootstrap.php --dont-report-useless-tests tests/FixtureTests.php
+
     - language: php
-      php: 8.0
+      php: 8.2
       install:
         # For PHP 8.0+, we need to ignore platform reqs as PHPUnit 7 is still used.
         - composer install --ignore-platform-reqs
       script:
-        - vendor/bin/phpunit
+        # For PHP 8.1+, we need to ignore the config file so that PHPUnit 7 doesn't try to read it and cause an error.
+        # Instead, we pass all required settings as part of the phpunit command.
+        - vendor/bin/phpunit --no-configuration --bootstrap=tests/bootstrap.php --dont-report-useless-tests tests/AllSniffs.php
+        - vendor/bin/phpunit --no-configuration --bootstrap=tests/bootstrap.php --dont-report-useless-tests tests/FixtureTests.php
+
     - language: php
-      php: 8.1
+      php: 8.3
       install:
         # For PHP 8.0+, we need to ignore platform reqs as PHPUnit 7 is still used.
         - composer install --ignore-platform-reqs
@@ -48,21 +45,63 @@ jobs:
         # Instead, we pass all required settings as part of the phpunit command.
         - vendor/bin/phpunit --no-configuration --bootstrap=tests/bootstrap.php --dont-report-useless-tests tests/AllSniffs.php
         - vendor/bin/phpunit --no-configuration --bootstrap=tests/bootstrap.php --dont-report-useless-tests tests/FixtureTests.php
+
+    - language: node_js
+      node_js: 20
+      install:
+        - npm ci
+        - cd packages/eslint-config-humanmade
+        - npm ci
+        - cd ../..
+      script:
+        - npm run test:eslint
+
     - language: node_js
-      node_js: 16
+      node_js: 20
       install:
-        - npm install
+        - npm ci
+        - cd packages/stylelint-config
+        - npm ci
+        - cd ../..
+      script:
+        - npm run test:stylelint
+
+    - language: node_js
+      node_js: 22
+      install:
+        - npm ci
         - cd packages/eslint-config-humanmade
-        - npm install --legacy-peer-deps
+        - npm ci
         - cd ../..
       script:
         - npm run test:eslint
+
+    - language: node_js
+      node_js: 22
+      install:
+        - npm ci
+        - cd packages/stylelint-config
+        - npm ci
+        - cd ../..
+      script:
+        - npm run test:stylelint
+
+    - language: node_js
+      node_js: 24
+      install:
+        - npm ci
+        - cd packages/eslint-config-humanmade
+        - npm ci
+        - cd ../..
+      script:
+        - npm run test:eslint
+
     - language: node_js
-      node_js: 16
+      node_js: 24
       install:
-        - npm install
+        - npm ci
         - cd packages/stylelint-config
-        - npm install
+        - npm ci
         - cd ../..
       script:
         - npm run test:stylelint

HM/Sniffs/Debug/ESLintSniff.php

@@ -18,9 +18,10 @@ class ESLintSniff implements Sniff {
 	/**
 	 * A list of tokenizers this sniff supports.
 	 *
+	 * Note: Disabled due to PHP_CodeSniffer 4.0 deprecation of CSS/JS scanning.
 	 * @var array
 	 */
-	public $supportedTokenizers = [ 'JS' ];
+	public $supportedTokenizers = [];
 
 	/**
 	 * ESLint configuration file path.