Changeset 59848

Timestamp:

02/20/2025 08:00:20 PM (2 months ago)

Author:

jorbin

Message:

Build/Test: Fix JavaScript linting scripts

wp-prettier was out of date and no longer compatible with wp-scripts. Additionally, many generated files and plugins were not being properly ignored which could cause linting to take so long that it was basically unusable.

Props ankitkumarshah, jorbin.
See #62935.

Location:

trunk

Files:

• .eslintignore (modified) (2 diffs)
• package-lock.json (modified) (3 diffs)
• package.json (modified) (1 diff)

trunk/.eslintignore

@@ -1 @@
-# Files and folders related to build/test tools
+# Files and folders related to build/test tools including generated files
 /build
 /node_modules
@@ -5 +5 @@
 /vendor
 /tools
+/jsdoc
+/artifacts
+/coverage
+.cache/*
+/src/wp-includes/blocks/**/*.js
+/src/wp-includes/blocks/**/*.js.map
+/src/wp-admin/js
+/src/wp-includes/js
 
 # Excluded files and folders based on `jsdoc.conf.json` exclusions
 /src/js/_enqueues/vendor
 
-# Webpack built files
-/src/wp-includes/js/media-*
+# Themes
+src/wp-content/themes
 
-# Themes
-src/wp-content/themes/
+# Files and folders that get created in wp-content
+/src/wp-content/plugins
+/src/wp-content/mu-plugins
+/src/wp-content/upgrade
+/src/wp-content/uploads

trunk/package-lock.json

@@ -142 +142 @@
                 "install-changed": "1.1.0",
                 "postcss": "8.4.49",
-                "prettier": "npm:wp-prettier@2.6.2",
+                "prettier": "npm:wp-prettier@3.0.3",
                 "qunit": "~2.23.1",
                 "react-refresh": "0.14.0",
@@ -9450 +9450 @@
             }
         },
-        "node_modules/@wordpress/scripts/node_modules/prettier": {
-            "name": "wp-prettier",
-            "version": "3.0.3",
-            "resolved": "https://registry.npmjs.org/wp-prettier/-/wp-prettier-3.0.3.tgz",
-            "integrity": "sha512-X4UlrxDTH8oom9qXlcjnydsjAOD2BmB6yFmvS4Z2zdTzqqpRWb+fbqrH412+l+OUXmbzJlSXjlMFYPgYG12IAA==",
-            "dev": true,
-            "bin": {
-                "prettier": "bin/prettier.cjs"
-            },
-            "engines": {
-                "node": ">=14"
-            },
-            "funding": {
-                "url": "https://github.com/prettier/prettier?sponsor=1"
-            }
-        },
         "node_modules/@wordpress/scripts/node_modules/read-pkg-up": {
             "version": "7.0.1",
@@ -28790 +28774 @@
         "node_modules/prettier": {
             "name": "wp-prettier",
-            "version": "2.6.2",
-            "resolved": "https://registry.npmjs.org/wp-prettier/-/wp-prettier-2.6.2.tgz",
-            "integrity": "sha512-AV33EzqiFJ3fj+mPlKABN59YFPReLkDxQnj067Z3uEOeRQf3g05WprL0RDuqM7UBhSRo9W1rMSC2KvZmjE5UOA==",
-            "dev": true
+            "version": "3.0.3",
+            "resolved": "https://registry.npmjs.org/wp-prettier/-/wp-prettier-3.0.3.tgz",
+            "integrity": "sha512-X4UlrxDTH8oom9qXlcjnydsjAOD2BmB6yFmvS4Z2zdTzqqpRWb+fbqrH412+l+OUXmbzJlSXjlMFYPgYG12IAA==",
+            "dev": true,
+            "license": "MIT",
+            "bin": {
+                "prettier": "bin/prettier.cjs"
+            },
+            "engines": {
+                "node": ">=14"
+            },
+            "funding": {
+                "url": "https://github.com/prettier/prettier?sponsor=1"
+            }
         },
         "node_modules/prettier-linter-helpers": {

trunk/package.json

@@ -63 +63 @@
         "install-changed": "1.1.0",
         "postcss": "8.4.49",
-        "prettier": "npm:wp-prettier@2.6.2",
+        "prettier": "npm:wp-prettier@3.0.3",
         "qunit": "~2.23.1",
         "react-refresh": "0.14.0",