From 90cb5f9a1f275b994a892e51f0e8e90cef83c3a5 Mon Sep 17 00:00:00 2001 From: silverwind Date: Wed, 17 Sep 2025 23:39:44 +0200 Subject: [PATCH] Clean up npm dependencies (#35508) - Remove `minimatch`, was inadvertently re-introduced in https://github.com/go-gitea/gitea/pull/35482 - Remove `postcss-nesting`, originally introduced in https://github.com/go-gitea/gitea/pull/29746 to support some fork, but I'm now of the opinion we should not ship dependencies that we don't use ourselves. - Remove `eslint-plugin-no-jquery`, plugin is currently disabled because of compat issues with eslint 9 (https://github.com/wikimedia/eslint-plugin-no-jquery/issues/311) --- package.json | 3 --- pnpm-lock.yaml | 41 ----------------------------------------- types.d.ts | 5 ----- webpack.config.ts | 3 --- 4 files changed, 52 deletions(-) diff --git a/package.json b/package.json index 832a629e71..0b2ce03354 100644 --- a/package.json +++ b/package.json @@ -38,7 +38,6 @@ "katex": "0.16.22", "mermaid": "11.11.0", "mini-css-extract-plugin": "2.9.4", - "minimatch": "10.0.3", "monaco-editor": "0.53.0", "monaco-editor-webpack-plugin": "7.1.0", "online-3d-viewer": "0.16.0", @@ -46,7 +45,6 @@ "perfect-debounce": "2.0.0", "postcss": "8.5.6", "postcss-loader": "8.2.0", - "postcss-nesting": "13.0.2", "sortablejs": "1.15.6", "swagger-ui-dist": "5.29.0", "tailwindcss": "3.4.17", @@ -90,7 +88,6 @@ "eslint-plugin-array-func": "5.0.2", "eslint-plugin-github": "6.0.0", "eslint-plugin-import-x": "4.16.1", - "eslint-plugin-no-jquery": "3.1.1", "eslint-plugin-no-use-extend-native": "0.7.2", "eslint-plugin-playwright": "2.2.2", "eslint-plugin-regexp": "2.10.0", diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml index 3ba3d91d55..5d40acd101 100644 --- a/pnpm-lock.yaml +++ b/pnpm-lock.yaml @@ -122,9 +122,6 @@ importers: mini-css-extract-plugin: specifier: 2.9.4 version: 2.9.4(webpack@5.101.3) - minimatch: - specifier: 10.0.3 - version: 10.0.3 monaco-editor: specifier: 0.53.0 version: 0.53.0 @@ -146,9 +143,6 @@ importers: postcss-loader: specifier: 8.2.0 version: 8.2.0(postcss@8.5.6)(typescript@5.9.2)(webpack@5.101.3) - postcss-nesting: - specifier: 13.0.2 - version: 13.0.2(postcss@8.5.6) sortablejs: specifier: 1.15.6 version: 1.15.6 @@ -273,9 +267,6 @@ importers: eslint-plugin-import-x: specifier: 4.16.1 version: 4.16.1(@typescript-eslint/utils@8.43.0(eslint@9.35.0(jiti@2.5.1))(typescript@5.9.2))(eslint-import-resolver-node@0.3.9)(eslint@9.35.0(jiti@2.5.1)) - eslint-plugin-no-jquery: - specifier: 3.1.1 - version: 3.1.1(eslint@9.35.0(jiti@2.5.1)) eslint-plugin-no-use-extend-native: specifier: 0.7.2 version: 0.7.2(eslint@9.35.0(jiti@2.5.1)) @@ -475,12 +466,6 @@ packages: '@csstools/css-parser-algorithms': ^3.0.5 '@csstools/css-tokenizer': ^3.0.4 - '@csstools/selector-resolve-nested@3.1.0': - resolution: {integrity: sha512-mf1LEW0tJLKfWyvn5KdDrhpxHyuxpbNwTIwOYLIvsTffeyOf85j5oIzfG0yosxDgx/sswlqBnESYUcQH0vgZ0g==} - engines: {node: '>=18'} - peerDependencies: - postcss-selector-parser: ^7.0.0 - '@csstools/selector-specificity@5.0.0': resolution: {integrity: sha512-PCqQV3c4CoVm3kdPhyeZ07VmBRdH2EpMFA/pd9OASpOEC3aXNGoqPDAZ80D0cLpMBxnmk0+yNhGsEx31hq7Gtw==} engines: {node: '>=18'} @@ -2582,11 +2567,6 @@ packages: peerDependencies: eslint: ^3 || ^4 || ^5 || ^6 || ^7 || ^8 || ^9 - eslint-plugin-no-jquery@3.1.1: - resolution: {integrity: sha512-LTLO3jH/Tjr1pmxCEqtV6qmt+OChv8La4fwgG470JRpgxyFF4NOzoC9CRy92GIWD3Yjl0qLEgPmD2FLQWcNEjg==} - peerDependencies: - eslint: '>=8.0.0' - eslint-plugin-no-only-tests@3.3.0: resolution: {integrity: sha512-brcKcxGnISN2CcVhXJ/kEQlNa0MEfGRtwKtWA16SkqXHKitaKIMrfemJKLKX1YqDU5C/5JY3PvZXd5jEW04e0Q==} engines: {node: '>=5.0.0'} @@ -3841,12 +3821,6 @@ packages: peerDependencies: postcss: ^8.2.14 - postcss-nesting@13.0.2: - resolution: {integrity: sha512-1YCI290TX+VP0U/K/aFxzHzQWHWURL+CtHMSbex1lCdpXD1SoR2sYuxDu5aNI9lPoXpKTCggFZiDJbwylU0LEQ==} - engines: {node: '>=18'} - peerDependencies: - postcss: ^8.4 - postcss-resolve-nested-selector@0.1.6: resolution: {integrity: sha512-0sglIs9Wmkzbr8lQwEyIzlDOOC9bGmfVKcJTaxv3vMmd3uo4o4DerC3En0bnmgceeql9BfC8hRkp7cg0fjdVqw==} @@ -4864,10 +4838,6 @@ snapshots: '@csstools/css-parser-algorithms': 3.0.5(@csstools/css-tokenizer@3.0.4) '@csstools/css-tokenizer': 3.0.4 - '@csstools/selector-resolve-nested@3.1.0(postcss-selector-parser@7.1.0)': - dependencies: - postcss-selector-parser: 7.1.0 - '@csstools/selector-specificity@5.0.0(postcss-selector-parser@7.1.0)': dependencies: postcss-selector-parser: 7.1.0 @@ -7120,10 +7090,6 @@ snapshots: safe-regex-test: '@nolyfill/safe-regex-test@1.0.44' string.prototype.includes: '@nolyfill/string.prototype.includes@1.0.44' - eslint-plugin-no-jquery@3.1.1(eslint@9.35.0(jiti@2.5.1)): - dependencies: - eslint: 9.35.0(jiti@2.5.1) - eslint-plugin-no-only-tests@3.3.0: {} eslint-plugin-no-use-extend-native@0.7.2(eslint@9.35.0(jiti@2.5.1)): @@ -8452,13 +8418,6 @@ snapshots: postcss: 8.5.6 postcss-selector-parser: 6.1.2 - postcss-nesting@13.0.2(postcss@8.5.6): - dependencies: - '@csstools/selector-resolve-nested': 3.1.0(postcss-selector-parser@7.1.0) - '@csstools/selector-specificity': 5.0.0(postcss-selector-parser@7.1.0) - postcss: 8.5.6 - postcss-selector-parser: 7.1.0 - postcss-resolve-nested-selector@0.1.6: {} postcss-safe-parser@6.0.0(postcss@8.5.6): diff --git a/types.d.ts b/types.d.ts index aa98d45465..5b01c2c761 100644 --- a/types.d.ts +++ b/types.d.ts @@ -17,11 +17,6 @@ declare module 'eslint-plugin-github' { const plugin: Eslint.Plugin; export = plugin; } -declare module 'eslint-plugin-no-jquery' { - import type {Eslint} from 'eslint'; - const plugin: Eslint.Plugin; - export = plugin; -} declare module '@eslint-community/eslint-plugin-eslint-comments' { import type {Eslint} from 'eslint'; const plugin: Eslint.Plugin; diff --git a/webpack.config.ts b/webpack.config.ts index 9d3020b0c7..35f55728a5 100644 --- a/webpack.config.ts +++ b/webpack.config.ts @@ -12,8 +12,6 @@ import {readFileSync, globSync} from 'node:fs'; import {env} from 'node:process'; import tailwindcss from 'tailwindcss'; import tailwindConfig from './tailwind.config.ts'; -import tailwindcssNesting from 'tailwindcss/nesting/index.js'; -import postcssNesting from 'postcss-nesting'; const {EsbuildPlugin} = EsBuildLoader; const {SourceMapDevToolPlugin, DefinePlugin, EnvironmentPlugin} = webpack; @@ -172,7 +170,6 @@ export default { options: { postcssOptions: { plugins: [ - tailwindcssNesting(postcssNesting({edition: '2024-02'})), tailwindcss(tailwindConfig), ], },